PR#2159: Include the priority name in the notification rather than its level - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

pagure

#2159 Include the priority name in the notification rather than its level

Merged Mar 24, 2017 by pingou. Opened Mar 23, 2017 by pingou.

priority_title into master

Download 2159.patch

pingou commented Mar 23, 2017

Fixes https://pagure.io/pagure/issue/2158

rebased

Mar 23, 2017

rebased

Mar 24, 2017

akasurde commented Mar 24, 2017

Might be good idea to get name for old_priority as well.

akasurde commented Mar 24, 2017

Use get with default value.

mbasti commented Mar 24, 2017

+100

pingou commented Mar 24, 2017

Yeah, I'm adding tests for this and figured the old priority might suffer from the same thing

rebased

Mar 24, 2017

pingou commented Mar 24, 2017

Ok, this is ready for review :)

puiterwijk commented Mar 24, 2017

Looks good to me.

pingou commented Mar 24, 2017

Thanks for the review :)

Pull-Request has been merged by pingou

Mar 24, 2017

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.