Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

MUMUP-2482 make priority a priority #207

Merged
merged 1 commit into from
Apr 4, 2016
Merged

MUMUP-2482 make priority a priority #207

merged 1 commit into from
Apr 4, 2016

Conversation

timlevett
Copy link
Contributor

Change sort to be priority (true first), then id desc.
http://goo.gl/QHgTCx

Also minor cleanup, features shouldn't be overwritten by default :)

@vertein
Copy link
Contributor

vertein commented Apr 4, 2016

👍

1 similar comment
@Doug-Reed
Copy link
Contributor

👍

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 35.404% when pulling 2a867df on sort-notifications-by-priority into e4e01a4 on master.

@timlevett timlevett merged commit fd4590a into master Apr 4, 2016
@timlevett timlevett deleted the sort-notifications-by-priority branch April 4, 2016 17:32
vertein pushed a commit to vertein/uportal-app-framework that referenced this pull request Jun 3, 2017
Merge pull request uPortal-Attic#207 from UW-Madison-DoIT/vanilla_ice
b59baca 
MUMUP-1830 : Set angularjs-portal up for working with master
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@timlevett @vertein @Doug-Reed @coveralls