Closed
Bug 1347275
Opened 9 years ago
Closed 9 years ago
Acknowledging an alert doesn't work
Categories
(Tree Management :: Perfherder, defect)
Tree Management
Perfherder
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wlach, Assigned: wlach)
References
Details
Attachments
(2 files)
It triggers a function that uses _.all, which is deprecated in the latest version of lodash
| Assignee | ||
Updated•9 years ago
|
Assignee: nobody → wlachance
Comment 1•9 years ago
|
||
| Assignee | ||
Updated•9 years ago
|
Attachment #8847248 -
Flags: review?(rwood)
Updated•9 years ago
|
Attachment #8847248 -
Flags: review?(rwood) → review+
Comment 2•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/6116f1218b647af868560f8c5e10f36ab5afa37e
Bug 1347275 - Fix acknowledging performance alerts (#2250)
Comment 3•9 years ago
|
||
| Assignee | ||
Comment 4•9 years ago
|
||
Comment on attachment 8847272 [details] [review]
[treeherder] wlach:1347275-2 > mozilla:master
I cherrypicked this to production already, but this could use a rubberstamp before I push to master. I think this should fix it all. I went through the lodash "changes" earlier today to verify that nothing still applied but apparently missed "removals".
Attachment #8847272 -
Flags: review?(rwood)
Comment 5•9 years ago
|
||
Comment on attachment 8847272 [details] [review]
[treeherder] wlach:1347275-2 > mozilla:master
LGTM
Attachment #8847272 -
Flags: review?(rwood) → review+
Comment 6•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/86d002918d4294b3fb80aae09cc45f3a40ab6cc9
Bug 1347275 - Another lodash fix for performance alerts (#2251)
| Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•