Rename &noignore=1 to &showall=1 (with support for the legacy format)

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Makes my eyes bleed less & may even (haha) reduce the stigma of a "hidden/ignored" job type.
(Assignee)

Comment 1

6 years ago
Created attachment 721290 [details] [diff] [review]
Patch v1

* s/noignore/showall/
* s/noIgnore/showAll/
* Adds a block to _parseParams() in Controller.js that maps the legacy format to the new one (ie: old URLs will still work, but will display the new parameter name in the address bar). See the block above it for how the existing code works with replacing &branch=buildbot_foo with &tree=foo (I didn't even know TBPL supported this!).
Attachment #721290 - Flags: review?(ryanvm)
(Assignee)

Updated

6 years ago
Summary: Rename &noignore=1 to &showall=1 → Rename &noignore=1 to &showall=1 (with support for the legacy format)
Attachment #721290 - Flags: review?(ryanvm) → review+
(Assignee)

Updated

6 years ago
Depends on: 849194
(Assignee)

Comment 3

6 years ago
In production :-)
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.