Skip to content

Disable add site import from WordPress.com if Studio is offline - #2284

Merged
bcotrim merged 2 commits into
trunkfrom
add-site-disable-wordpress-if-offline
Dec 24, 2025
Merged

Disable add site import from WordPress.com if Studio is offline#2284
bcotrim merged 2 commits into
trunkfrom
add-site-disable-wordpress-if-offline

Conversation

@bcotrim

@bcotrim bcotrim commented Dec 18, 2025

Copy link
Copy Markdown
Contributor

Related issues

  • N/A

Proposed Changes

  • Disable the option to "Import from WordPress.com" when adding a site if Studio is offline
Screenshot 2025-12-18 at 10 52 43

Testing Instructions

  • npm start
  • disable your network connection
  • add site
  • confirm option to import from WordPress.com is disable
  • enable network connection
  • confirm option to import from WordPress.com is enabled

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?
@bcotrim
bcotrim requested a review from a team December 18, 2025 10:56
@bcotrim bcotrim self-assigned this Dec 18, 2025

@katinthehatsite katinthehatsite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good and work as expected:

Image

We could potentially improve the offline message with more detail but the current version also works

@gcsecsey gcsecsey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and works well, the transition is pretty instantenous after toggling the connection.

CleanShot.2025-12-19.at.15.54.02.mp4
@bcotrim
bcotrim force-pushed the add-site-disable-wordpress-if-offline branch from 2889a67 to 6fc6fa6 Compare December 19, 2025 16:21
@bcotrim

bcotrim commented Dec 19, 2025

Copy link
Copy Markdown
Contributor Author

@katinthehatsite @gcsecsey thanks for the reviews!
I've updated the error messages, please let me know what you think.

@github-actions

Copy link
Copy Markdown
Contributor

📊 Performance Test Results

Comparing 4a4b7fc vs trunk

site-editor

Metric trunk 4a4b7fc Diff Change
load 6826.00 ms 10437.00 ms +3611.00 ms 🔴 52.9%

site-startup

Metric trunk 4a4b7fc Diff Change
siteCreation 14165.00 ms 12212.00 ms -1953.00 ms 🟢 -13.8%
siteStartup 5961.00 ms 5941.00 ms -20.00 ms 🟢 -0.3%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change

@gcsecsey gcsecsey left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for iterating on the message @bcotrim, it's much more informative now. 👍

Image
@bcotrim
bcotrim merged commit 53ed141 into trunk Dec 24, 2025
10 checks passed
@bcotrim
bcotrim deleted the add-site-disable-wordpress-if-offline branch December 24, 2025 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants