Closed
Bug 1440759
Opened 7 years ago
Closed 7 years ago
links from push timestamp don't preserve filters because the format in the url parameter changed, no filters active
Categories
(Tree Management :: Treeherder: Frontend, defect)
Tree Management
Treeherder: Frontend
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: aryx, Assigned: aryx)
References
Details
(Keywords: regression)
Attachments
(1 file)
Steps to reproduce:
1. Disable some of the circle filters left of the search box.
2. Reload the page (at the moment, the format of the links doesn't live update; also a regression afaik).
3. Copy the link from the timestamp of a push, e.g. https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=1b6ba1c15366fb08dc376b403210615505422966&filter-resultStatus=testfailed,busted,exception,retry,usercancel,runnable
4. Open that link in a new tab.
Actual result: All filters disabled.
Expected result: Filters preserved.
Caused by:
Angular did this different: https://github.com/mozilla/treeherder/blame/c422eb968dd9c21af8fd01c4ad657fa4a1967784/ui/js/controllers/main.js#L675
Docs: https://docs.angularjs.org/api/ng/service/$httpParamSerializer
Comment 1•7 years ago
|
||
Assignee | ||
Updated•7 years ago
|
Attachment #8953576 -
Flags: review?(cdawson)
Comment 2•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/dec0e3caa154c1233505c28cb73eb09a70a994ca
Bug 1440759 - Preserve active filters in links from push timestamp. r?camd (#3261)
Updated•7 years ago
|
Attachment #8953576 -
Flags: review?(cdawson) → review+
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
•