Closed Bug 1105741 Opened 10 years ago Closed 10 years ago

priority sort in new ui is backwards

Categories

(Release Engineering Graveyard :: Applications: Balrog (frontend), defect)

Product:
Release Engineering Graveyard
Component:
Applications: Balrog (frontend)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: peterbe)

References

Details

Priority is better sorted descending - we care about the highest priority things more than the lowest. The code that sets up the sorts is at https://github.com/mozilla/balrog-ui/blob/master/app/js/controllers/rules_controller.js#L36, though I'm not sure how to invert one of the fields.
PR https://github.com/mozilla/balrog-ui/pull/4
Assignee: nobody → peterbe
Status: NEW → ASSIGNED
Depends on: 1108573
Depends on: 1109115
In production and looking good!
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Commits pushed to master at https://github.com/mozilla/balrog https://github.com/mozilla/balrog/commit/1c28336dd1b5cf402c903c77c41b9d2a4878a1ce [balrog-ui] fixes bug 1105741 - priority sort in new ui is backwards https://github.com/mozilla/balrog/commit/79144c77cda01e6091d44970843933e3249f95ad [balrog-ui] Merge pull request #4 from peterbe/bug-1105741-priority-sort-in-new-ui-is-backwards bug 1105741: priority sort in new ui is backwards
Product: Release Engineering → Release Engineering Graveyard
You need to log in before you can comment on or make changes to this bug.