-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
FSE: Unable to save changes & repeated Update Design notice #29452
Comments
Reminds me of this one: #28271 I closed it because destroying and recreating my environment fixed the issue for me. |
Ah thanks for connecting the dots here. I have to ask though, you say this in your issue:
It worries me to close this out without investigating as ideally the Site Editor is stable enough at this point to be used across time. This was a more recent test site as well that I created on December 14, 2020. I'll close it out but curious what the thought process is as I imagine part of testing should be trying to snag problems like this. |
Sorry I did not necessarily mean you should close this one. In fact since it's happening again, I would agree it needs a little investigation. |
Ah! I made an assumption there. I'll re-open. Thanks again for connecting dots :) |
In testing out #28141, I encountered this issue again. It was then fixed again when I checked out trunk. Not quite sure what caused it, but as I had some dev environment issues with testing that PR (needing to |
This just happened to another test site of mine out of the blue using the same setup (GB version, WP version, TT1 blocks). I edited the 404 template before this happened. |
Were you able to look at the browser's Network dev tool and grab any info on which/how requests may be failing? |
Back in #28271 I did in fact record a video of the process when chatting with @david-szabo97: Screeny.Video.19.Jan.2021.at.10.03.24.movHowever note that I can't easily reproduce this anymore. For me it happens sporadically and mostly on older pull requests. |
Ah yes! Apologies. Forgot to share that here. network.calls.movI don't see anything noteworthy. |
This seems to have magically resolved itself at some point. I stopped using the test sites impacted by this, revisited them this week with the latest GB version, and everything works as expected now. I am closing this out a bit bummed the root problem wasn't determined but happy the issue is no longer present it seems. |
Description
When trying to edit and save within the site editor experience, I get stuck in a repeated "Update Design" flow that doesn't seem to result in changes actually being saved. This is a new bug that wasn't present when I've done prior testing so I'm not quite sure what's changed and I wasn't able to find any open/closed related issues. This is being done as part of work for the FSE Outreach program.
Step-by-step reproduction instructions
Please list the steps needed to reproduce the bug. For example:
Expected behaviour
I expect the Update Design notice to grey out when changes aren't made. I expect changes to stick upon saving them.
Actual behaviour
Update Design repeatedly shows as a call to action and the changes don't seem to save despite going through the saving flow.
Screenshots or screen recording (optional)
saving.failing.mov
Of note, this is very randomly inconsistent. For example, here's where a simple change to a post title stuck followed by the error recurring:
query.block.saving.mov
Here's me editing the post title in the same exact why but the strange saving behavior recurring:
same.flow.failing.mov
WordPress information
The text was updated successfully, but these errors were encountered: