Closed
Bug 1521484
Opened 7 years ago
Closed 7 years ago
calling 'get next: 10 / 20 / 50 pushes' loads them but doesn't add "fromchange <revision>" to 'Active Filters' bar anymore
Categories
(Tree Management :: Treeherder: Frontend, defect, P1)
Tree Management
Treeherder: Frontend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: aryx, Assigned: camd)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Calling 'get next: 10 / 20 / 50 pushes' loads them but doesn't add "fromchange <revision>" to 'Active Filters' bar anymore.
Steps to reproduce:
- Load a tree, e.g. https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cusercancel%2Cretry&group_state=expanded
- Go to the bottom and let it load 10 more pushes.
Actual result:
10 more pushes loaded but bar "Active Filters:" has no rule "fromchange". URL contains fromchange. - Reload the page.
Actual result:
"Active Filters:" contains "fromchange".
Comment 1•7 years ago
|
||
I can reproduce on production.
Bisecting locally, it seems to have started as of:
https://github.com/mozilla/treeherder/pull/4353
Comment 2•7 years ago
|
||
| Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(cdawson)
Updated•7 years ago
|
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
Comment 3•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/2d1c4adb7ca6164f5b4d0264ef3eddbbd377ceb6
Bug 1521484 - Fix fromchange showing in ActiveFiltersBar (#4492)
I forgot to add this and a few others to the list of filters
that should get inspected in the App.handleUrlChange.
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•