Studio: Ensure the site can't be deleted when syncing - #2254
Conversation
📊 Performance Test ResultsComparing d8fe9dc vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
ivan-ottinger
left a comment
There was a problem hiding this comment.
Thank you for the fix, Kat! I have observed one inconsistency where the Delete site... button in the Settings tab is disabled for the whole process, while the similar option in the sidebar dropdown gets disabled only for a small part of the process:
CleanShot.2025-12-17.at.13.42.06.mp4
👀 |
|
Great catch @ivan-ottinger - it should be fixed now. Would you be able to give it another try? Thank you! |
There was a problem hiding this comment.
Great catch @ivan-ottinger - it should be fixed now. Would you be able to give it another try? Thank you!
Thanks for the fix, Kat. The changes work as expected (bot delete buttons are now disabled in the same way). The code looks generally good to me. I have just shared one suggestion and noticed that tests are failing.
gcsecsey
left a comment
There was a problem hiding this comment.
Thanks @katinthehatsite for working on this! I tested both pushing and pulling and now I confirm not being able to delete the site for the whole duration, either from the sidebar or from the Settings tab.
CleanShot.2025-12-17.at.17.24.18.mp4
…local-site-cannot-be-deleted-during-sync
ivan-ottinger
left a comment
There was a problem hiding this comment.
Thanks for the updates, Kat! All is looking good to me now. ✅
Related issues
Fixes STU-1127
Proposed Changes
This PR ensures that the local site can not be deleted when syncing to WP.com (either pulling or pushing).
Testing Instructions
SynctabSettingstabDeletesite button is disabled and the site can not be deletedDelete sitebutton is disabledPre-merge Checklist