alerts without extensions don't render |
|||
Issue descriptionProblem with Sheriff-o-Matic-staging: If there are alerts without extensions, they don't render due to a null in _haveSuspectCLs This manifests as a visible heading for "Stale Masters" but no alerts underneath it.
,
Aug 3 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/93c2512cd32375ba8859d9cae310a0de18e6afe7 commit 93c2512cd32375ba8859d9cae310a0de18e6afe7 Author: seanmccullough <seanmccullough@google.com> Date: Wed Aug 03 18:48:21 2016 [som] Render alerts that don't have extensions. BUG= 634037 Review-Url: https://codereview.chromium.org/2203003006 [modify] https://crrev.com/93c2512cd32375ba8859d9cae310a0de18e6afe7/go/src/infra/appengine/sheriff-o-matic/elements/som-extension-build-failure.html [modify] https://crrev.com/93c2512cd32375ba8859d9cae310a0de18e6afe7/go/src/infra/appengine/sheriff-o-matic/test/som-extension-build-failure-test.html
,
Aug 3 2016
,
Aug 4 2016
Just deployed a new version with Sean's fix for this, so marking this as Fixed. :) |
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Aug 3 2016