Skip to content
This repository has been archived by the owner on Oct 3, 2023. It is now read-only.

nicer error reporting to editors #411

Open
jacqui opened this issue Apr 6, 2022 · 0 comments
Open

nicer error reporting to editors #411

jacqui opened this issue Apr 6, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@jacqui
Copy link
Contributor

jacqui commented Apr 6, 2022

Instead of displaying an error like the following to someone trying to edit an article, catch it and give a nicer explanation that makes sense:

Re-Publish: An error occurred: ScriptError: Exception: API call to docs.documents.get failed with error: Empty response

Suggestions welcome, but one idea (from me) is:

Re-Publish: Sorry, Google Docs seems to be having an issue at the moment. Try again in a few minutes. If this problem persists, contact #tech-support on tinynews.slack.com.

I'm happy to make the code change - some input on how best to convey something's gone wrong (that might not be in our control) would be great though!

Also, note: if possible we should catch types of error and display a message that makes sense for each. In the above example, catch an API error in Google Docs/Drive.

Another type of error we could handle better: the authorization issue (ScriptError: Authorization). As Tyler pointed out, that one isn’t about a Google Docs problem, that’s about the user being logged in incorrectly

@jacqui jacqui added enhancement New feature or request question Further information is requested labels Apr 6, 2022
@chelsea-tnc chelsea-tnc removed the question Further information is requested label Jul 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants