New issue
Advanced search Search tips

Issue 667985 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

Make alert sections stay in place

Project Member Reported by zhangtiff@chromium.org, Nov 23 2016

Issue description

Based on: https://docs.google.com/document/d/1rs4GeM-BxEyoj5L1nyEJPATTdPWVoXUjqUth71AKYSo/edit#

I think the current behavior of alert sections is to reorder based on the sort order of the alerts within the section. For example, if all alerts in a section are snoozed, they are moved below other sections. 

We should sort by severity titles first then sort within sections second. Perhaps if all alerts in a section are snoozed, we can collapse that section instead of moving it to the bottom or something. 
 
Owner: zhangtiff@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 29 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/c6421532959e101bacd1ee3f479b1ade7d2ff329

commit c6421532959e101bacd1ee3f479b1ade7d2ff329
Author: Tiffany Zhang <zhangtiff@google.com>
Date: Tue Nov 29 00:20:10 2016

SoM: Make severity sections for alerts stay in place.

BUG= 667985 

Change-Id: Idc48dabcc7003f0b9ed77a8401471edaaae975be
Reviewed-on: https://chromium-review.googlesource.com/414525
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>
Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org>

[modify] https://crrev.com/c6421532959e101bacd1ee3f479b1ade7d2ff329/go/src/infra/appengine/sheriff-o-matic/elements/som-app.html

This will be deployed tomorrow. 
Status: Fixed (was: Started)

Sign in to add a comment