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

Site Editor: Global Styles are not persisted on WP.com #50358

Closed
7 tasks done
vindl opened this issue Feb 22, 2021 · 7 comments
Closed
7 tasks done

Site Editor: Global Styles are not persisted on WP.com #50358

vindl opened this issue Feb 22, 2021 · 7 comments
Assignees
Labels
[Closed] Fixed Issues explicitly fixed with PRs. Do not use for no longer reproducible issues. [Goal] Full Site Editing [Size] M Medium sized issue [Type] Bug

Comments

@vindl
Copy link
Member

vindl commented Feb 22, 2021

The saving flow for Global Styles is not working properly now on Dotcom. The preview in editor works fine but there are two potentially related issues when you attempt to update/save the changes:

Similar already reported issues that probably have the same root cause:

P2 comment with context of what might be causing the problem: paYE8P-Km-p2#comment-627

@vindl
Copy link
Member Author

vindl commented Feb 22, 2021

There is some more context about what might be going on here: paYE8P-Km-p2#comment-627

@autumnfjeld
Copy link
Contributor

autumnfjeld commented Mar 23, 2021

Some points of investigation from Ganon estimation discussion:

  • Could jetpack be the culprit
  • Could public-api manipulation be the culprit
  • Changes to 3rd party cookie requests
  • Custom post types
  • Start with tracing data

Start with at least Size M due to unknowns/need to investigate.

@p-jackson p-jackson self-assigned this Mar 25, 2021
@simison
Copy link
Member

simison commented Mar 26, 2021

Related convo in paYE8P-PE-p2

@p-jackson
Copy link
Member

p-jackson added the [Status] Blocked / Hold label

Waiting for more discussion on paYE8P-PE-p2

@simison
Copy link
Member

simison commented Apr 21, 2021

PR: WordPress/gutenberg#30888

@p-jackson
Copy link
Member

WordPress/gutenberg#30888 has merged, we just need to verify the bug has been fixed once the GB plugin is updated on wpcom

@p-jackson
Copy link
Member

Verified, thanks @vindl! pbAok1-26m-p2#comment-4407

@p-jackson p-jackson added the [Closed] Fixed Issues explicitly fixed with PRs. Do not use for no longer reproducible issues. label May 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Closed] Fixed Issues explicitly fixed with PRs. Do not use for no longer reproducible issues. [Goal] Full Site Editing [Size] M Medium sized issue [Type] Bug
Projects
None yet
Development

No branches or pull requests

4 participants