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

[Dev] Fix issue when saving via the "Unable to save" message showing "Saved" when it's actually failing to save #2830

Closed
1 task
tbolt opened this issue Feb 1, 2021 · 0 comments · Fixed by #2927
Assignees
Labels
bug needs triage, then squashing Development Issues for the dev team resolve medium Quality Quality Assurance and Quality Control sev3 Experience is flawed but not unusable.

Comments

@tbolt
Copy link
Contributor

tbolt commented Feb 1, 2021

Bug Found in #2848
In my testing I was able to force the "Unable to save" message by shutting down the backend. Then when clicking the "Save" button provided by that alert message, it would spin for a minute and then say "Saved." However, this is not actually saving and there is an error thrown in the console (plus the backend is turned off)

Bug -
Notice in this screenshot the error connection refused and the SAVE_APD action failed. Yet the UI shows it was successfully saved.
image.png

This task is done when…

  • The UI reflects the state of the system accurately
@tbolt tbolt added bug needs triage, then squashing Development Issues for the dev team resolve labels Feb 1, 2021
@thetif thetif added medium sev3 Experience is flawed but not unusable. labels Feb 2, 2021
@nicholeweems nicholeweems added this to the 02/22/2021 - 03/05/2021 milestone Feb 18, 2021
@eStar386 eStar386 added the Quality Quality Assurance and Quality Control label Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug needs triage, then squashing Development Issues for the dev team resolve medium Quality Quality Assurance and Quality Control sev3 Experience is flawed but not unusable.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants