Better error handling on the new SOM frontend |
|||||
Issue descriptionRight now, we basically ignore errors, and if any of the AJAX requests which hit the server fail, the user doesn't really know at all. They have to open up the developer console to see these errors. We really should show these errors to the user, and do it in a nice way.
,
Jun 29 2016
,
Jul 7 2016
I will work on this.
,
Jul 26 2016
,
Jul 26 2016
Issue 631105 has been merged into this issue.
,
Aug 4 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/3b2d7edcd7682aca8fe42c4b704e5463dcaad485 commit 3b2d7edcd7682aca8fe42c4b704e5463dcaad485 Author: zhangtiff <zhangtiff@chromium.org> Date: Thu Aug 04 17:28:02 2016 SoM: Alerts error handling. BUG= 621156 Review-Url: https://codereview.chromium.org/2200943002 [modify] https://crrev.com/3b2d7edcd7682aca8fe42c4b704e5463dcaad485/go/src/infra/appengine/sheriff-o-matic/elements/som-app.html [modify] https://crrev.com/3b2d7edcd7682aca8fe42c4b704e5463dcaad485/go/src/infra/appengine/sheriff-o-matic/test/som-app-test.html
,
Aug 4 2016
The latest revision here fixed error handling for alert loading. There was another CL I did earlier that never made it here that fixed error handling for submitting annotations. I think that covers us on AJAX errors. The bug queue already has error handling. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by benhenry@chromium.org
, Jun 29 2016