#1918 Always include a blank priority field
Merged by pingou. Opened by pingou.
fix_priority  into  master

Download 1918.patch

This will make an empty priority row in the html which is then used
as template for when adding another row or the default priorities

Fixes https://pagure.io/pagure/issue/1917
Fixes https://pagure.io/pagure/issue/1915

Hotfixed in prod :)

rebased

rebased

note that it doesn't "always" include it, only if there's no other priorities.
I think it would be good to add a test for this behavior.

1 new commit added

  • Add unit-tests checking the default rendering of some fields in settings

rebased

rebased

Ok, got the green light for this one from @puiterwijk on IRC, so let's merge :)

Pull-Request has been merged by pingou

Metadata