Move createSiteFromRemote from feature flag to beta flag - #2078
Conversation
📊 Performance Test ResultsComparing 83d497e vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
Co-authored-by: Wojtek Naruniec <wojtek.naruniec@automattic.com>
Co-authored-by: Wojtek Naruniec <wojtek.naruniec@automattic.com>
| description: '"studio site" command to manage local sites from terminal', | ||
| }, | ||
| createSiteFromRemote: { | ||
| label: 'Create a site from existing remote site', |
katinthehatsite
left a comment
There was a problem hiding this comment.
The changes look good and work as expected 👍 I left one minor comment but feel free to proceed without implementing it.
ivan-ottinger
left a comment
There was a problem hiding this comment.
The changes look good and work as expected. The beta feature selection works correctly:
CleanShot.2025-11-20.at.10.23.24.mp4
The remote site gets pulled right-away (I just did not record it).
I only noticed one small issue of extra line.
|
I suggest following a different pattern to implement this (same concern as I raised in #2083 (comment)). The RTK Query See Basically, whenever we need to dispatch a "load data" action at app start (like we do here), that's an opportunity to consider using RTK Query instead. |

Related issues
Proposed Changes
Testing Instructions
npm startbeta-flag-create-site-from-remote.mp4
Pre-merge Checklist