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

In case of a save error, use notify.error, not a full screen fatal page #12381

Merged

Conversation

stacey-gammon
Copy link
Contributor

Don't use the fatal screen when a save fails, only use the notification error banner.

@alexfrancoeur
Copy link

@stacey-gammon any chance we can get this backported to 5.5 for the new demo site?

@alexfrancoeur
Copy link

Also, this was seen when attempting to save a visualization as well. Would this fix resolve the viz side of this as well?

Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stacey-gammon
Copy link
Contributor Author

stacey-gammon commented Jun 19, 2017

Will do @alexfrancoeur - and yes, this covers visualizations as well.

@alexfrancoeur
Copy link

Thank you @stacey-gammon!!

@stacey-gammon stacey-gammon deleted the fix/error-not-fatal-on-save branch October 24, 2017 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants