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

Provide some safety fallback UX when "Updating failed" #4590

Closed
danielbachhuber opened this issue Jan 19, 2018 · 12 comments
Closed

Provide some safety fallback UX when "Updating failed" #4590

danielbachhuber opened this issue Jan 19, 2018 · 12 comments
Labels
Backwards Compatibility Issues or PRs that impact backwards compatability [Feature] Offline Related to offline editing [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Enhancement A suggestion for improvement.
Milestone

Comments

@danielbachhuber
Copy link
Member

As a Gutenberg user, seeing this error message is quite terrifying:

image

Specifically, my emotional state is:

"Updating failed"?! Does this mean I'm going to lose all of the content I published?! How do I get back to safely updating?!

It would be ideal to have some secondary UX in this error state ("save your post to your browser" or similar).

The specific issue I ran into is CloudFlare deciding to block my PUT requests.

@danielbachhuber danielbachhuber changed the title Provide some safety fallback when "Updating failed" Provide some safety fallback UX when "Updating failed" Jan 19, 2018
@aduth
Copy link
Member

aduth commented Jan 19, 2018

The specific issue I ran into is CloudFlare deciding to block my PUT requests.

Unrelated to the UX concern you mention, but noting that this specific issue should be resolved as of #4396.

@jorgefilipecosta
Copy link
Member

Maybe we can use the new copy all content mechanism and offer the user a link that allows copying all content to the clipboard so they feel safer.

@danielbachhuber danielbachhuber added Needs Design Needs design efforts. [Type] Enhancement A suggestion for improvement. labels Jan 19, 2018
@karmatosed
Copy link
Member

karmatosed commented Jan 22, 2018

One thing discussed before is adding context to errors. If we know what the error is, let's say it here. Design wise we don't have to change the color or nature as it is an error message. What we can do is be a lot more human in our content.

A further addition could be yes adding a 'would you like to copy the content' or just to maybe offer to create a new post. Some way they get to the end result of publishing, without having to go through the entire process again or at least less of the process.

If the message is temporary and the update may succeed again, a simple "would you like to try again", would be great.

To summarise:

  • If we know the context lets add the reason why it failed to the error message - let's extend that to any messages.
  • If we can offer a fast path to publishing success, let's do that and allow steps to be skipped.

@karmatosed karmatosed removed the Needs Design Needs design efforts. label Jan 22, 2018
@ghost
Copy link

ghost commented Jan 22, 2018

Showing more specific error messages would also be useful for server side validation : #4063

@karmatosed karmatosed added this to the Merge Proposal: Editor milestone Apr 27, 2018
@mtias
Copy link
Member

mtias commented Jun 22, 2018

Related #7367.

@tofumatt tofumatt added the [Feature] Offline Related to offline editing label Jun 27, 2018
@mtias mtias modified the milestones: Merge: Editor, WordPress 5.0 Oct 3, 2018
@mtias mtias added the [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later label Oct 3, 2018
@designsimply designsimply added the Backwards Compatibility Issues or PRs that impact backwards compatability label Nov 8, 2018
@paaljoachim
Copy link
Contributor

What is the status of this issue? I can not remember seeing the error in a while.

Greg @gziolo

@gziolo
Copy link
Member

gziolo commented Jan 22, 2021

@paaljoachim, I don't know this issue.

@paaljoachim
Copy link
Contributor

Thanks for the response Greg!

@paaljoachim
Copy link
Contributor

paaljoachim commented Feb 15, 2021

Thoughts on this issue?
@talldan @tellthemachines @mcsf
(Making my best guess whom I can ping that might know something about the update/save process.)

@talldan
Copy link
Contributor

talldan commented Feb 16, 2021

Not sure that it's much of an issue any longer. The underlying reason for the error seems to have been fixed.

There's now also a process that backs up posts to local browser storage, which offers another recovery option.

To add to the previous comment (#4590 (comment)), offering a Copy All Content option seems like a reasonable thing to do for errors relating to save.

Not sure what else can be done, usually when saving fails there's a good reason that can't be easily solved.

@mcsf
Copy link
Contributor

mcsf commented Feb 16, 2021

I'm up for closing this.

@jorgefilipecosta
Copy link
Member

Closing this issue as there are no additional action items.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backwards Compatibility Issues or PRs that impact backwards compatability [Feature] Offline Related to offline editing [Priority] Low Used to indicate that the issue at hand isn't a top priority to address and can be handled later [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests