Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Interface preference should be synced between production and staging sites #91712

Closed
mrfoxtalbot opened this issue Jun 12, 2024 · 11 comments
Closed
Assignees
Labels
[Feature Group] Developer & Contractor Tools Features and tools designed for developers and contractors working on WordPress.com sites. [Feature] Staging Sites All tools and settings that allow you to create staging sites for Atomic sites. [Platform] Atomic [Pri] Normal [Product] WordPress.com All features accessible on and related to WordPress.com. Triaged To be used when issues have been triaged. [Type] Bug

Comments

@mrfoxtalbot
Copy link

Quick summary

If I create a staging site for a classic interface site, the newly created staging site has the default interface option, not the classic one:

338131753-612e420f-1bda-472e-bdb6-39f59842dc1e

Screenshot 2024-06-12 at 11 55 04

Conversely, if I push a Default interface staging site into production replacing a Classic interface one, the production site will stay Classic.

In other words, the preference is not syncing at all.

Steps to reproduce

  1. Set a site to "Classic Interface"
  2. Create a staging site

What you expected to happen

The staging site should also be "Classic"

What actually happened

The staging site was set to default

Impact

All

Available workarounds?

Yes, easy to implement

Platform (Simple and/or Atomic)

Atomic

Logs or notes

No response

@mrfoxtalbot mrfoxtalbot added [Type] Bug Needs triage Ticket needs to be triaged [Feature] Staging Sites All tools and settings that allow you to create staging sites for Atomic sites. [Product] WordPress.com All features accessible on and related to WordPress.com. [Feature Group] Developer & Contractor Tools Features and tools designed for developers and contractors working on WordPress.com sites. labels Jun 12, 2024
@BogdanUngureanu BogdanUngureanu self-assigned this Jun 12, 2024
@mrfoxtalbot mrfoxtalbot added Triaged To be used when issues have been triaged. and removed Needs triage Ticket needs to be triaged labels Jun 18, 2024
@BogdanUngureanu
Copy link
Contributor

Looks like the issue is not reproducing anymore (although yesterday it was). @mrfoxtalbot feel free to re-open the issue if you're still reproducing it.

@mrfoxtalbot
Copy link
Author

I re-tested this just not and the issue is still there, @BogdanUngureanu.

  1. Set a site to Classic Interface
  2. Spin a staging site
  3. Notice how the staging site is set to Default

Screenshot 2024-06-21 at 07 54 02

Can we check how the interface preference is stored and confirm if this setting is synchronized? During pre-launch, this was set via an option named wpcom_classic_early_release. Are we still using this in production @allilevine? Thanks!

@mrfoxtalbot mrfoxtalbot reopened this Jun 21, 2024
@mrfoxtalbot
Copy link
Author

mrfoxtalbot commented Jun 21, 2024

Here are two more screenshots to illustrate the issue:

Productions site (Classic):

Screenshot 2024-06-21 at 08 09 52

Staging site (Default):

Screenshot 2024-06-21 at 08 10 03

@BogdanUngureanu
Copy link
Contributor

Currently I'm focusing on T-Rex Readymade templates and I believe some spare time to look at the Production-Staging site sync issue again next week. If nobody takes the issue in the meantime, I'll give it another spin (weird that I wasn't able to reproduce it 🤔 ).

@BogdanUngureanu BogdanUngureanu removed their assignment Jun 24, 2024
@mrfoxtalbot
Copy link
Author

Thank you @BogdanUngureanu. Maybe @Automattic/explorers could a look? 🙏

@allilevine
Copy link
Member

There's definitely something weird going on here. I just created a Staging Site and in Calypso, it's set to Default view, but in wp-admin, it's set to Classic view.

Screen Shot 2024-06-24 at 3 30 28 PM Screen Shot 2024-06-24 at 3 30 15 PM

@mrfoxtalbot I see this is in Unassigned on the Untangling board, which is the best place for it to get picked up!

@BogdanUngureanu BogdanUngureanu self-assigned this Jul 1, 2024
@mrfoxtalbot
Copy link
Author

I see @BogdanUngureanu is on it already.

Thank you both!

@BogdanUngureanu
Copy link
Contributor

BogdanUngureanu commented Jul 5, 2024

I've opened D154352-code which should fix these two problems:

  • When the staging site is created, we now import the Admin interface preference from the production site
  • On Sync, the Admin interface will also be synched (for both types of sync - Prod to Staging and vice-versa)

Since I'm not super familiar with this area of the code, any feedback on the diff is more than welcomed. :)

@mrfoxtalbot
Copy link
Author

Thank you, @BogdanUngureanu. Will you PR fit this problem too?:

@BogdanUngureanu
Copy link
Contributor

@mrfoxtalbot no, we just sync the Admin interface option, but I assume the fix would be similar (I haven't checked how the module settings are saved).

Sorry for the long response time, I got involved with other tasks and missed your messages.

@BogdanUngureanu
Copy link
Contributor

The fix has landed on production with D154352-code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature Group] Developer & Contractor Tools Features and tools designed for developers and contractors working on WordPress.com sites. [Feature] Staging Sites All tools and settings that allow you to create staging sites for Atomic sites. [Platform] Atomic [Pri] Normal [Product] WordPress.com All features accessible on and related to WordPress.com. Triaged To be used when issues have been triaged. [Type] Bug
Development

No branches or pull requests

3 participants