Closed
Bug 1229101
Opened 10 years ago
Closed 10 years ago
Changing alert nomenclature
Categories
(Tree Management :: Perfherder, defect)
Tree Management
Perfherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wlach, Assigned: wlach)
Details
Attachments
(1 file)
After some experimentation with the UI, I think it makes sense to give the alert summary the title "alert". Correspondingly, the individual alerts are probably better thought of as "detected changes".
Going to attach a patch to change the frontend. We should change the backend to reflect that, but there's probably no huge rush at this point.
|
Assignee | |
Comment 1•10 years ago
|
||
See the github PR for a screenshot. I think this makes more sense than the alerts/alerts summary metaphor I initially went with.
Attachment #8693711 -
Flags: review?(jmaher)
|
||
Comment 2•10 years ago
|
||
Comment on attachment 8693711 [details] [review]
PR
nice and simple
Attachment #8693711 -
Flags: review?(jmaher) → review+
|
||
Comment 3•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/edec73c6140559760a67f49be3f56005578fd19d
Bug 1229101 - Change alert titling
* Alert Summary -> Alert
* Alert -> Detected Changes
* Include id # in summary (for people to look up and reference later)
|
|
Assignee | |
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•