This PR adds option to add orphaning reason to package on dist-git. User can choose from various options and write additional info if needed.
This is implementing the changes in Pagure needed for https://pagure.io/pagure-dist-git/pull-request/118.
I tried to run the pagure with dist-git on my machine, but without luck, so the changes here are untested :-(
Signed-off-by: Michal Konečný mkonecny@redhat.com
:thumbsup:
rebased onto 775019d98db3c2cac537a2f2a5b1d1cd0ae7190f
Something is broken aroung the dropdown:
rebased onto a1b1acf265ce9ee4946f8fce27c6116fc21a20bb
I fixed the dropdown, there are still some styling issues. I will look at them tomorrow.
1 new commit added
Fix form
Add dynamic elements
rebased onto a6aa28a00c151172b581fcbd492fe4845285d740
Should be finally ready for merging
@zlopez Could you please squash all the commits?
rebased onto 9abb8833f586787383a94db59cf3046a500a72b0
@ngompa squashed
rebased onto 326845fd07661d2327e057712655e277182ab79e
Technically working but I'd like to change the UI around this, I'll send a patch
Here is my proposed patch, let me know what you think of it
Here are a couple of screenshots:
This looks much nicer, I will apply the patch
rebased onto 07855845e355bd1b813c721409f22bfcece4f90c
@pingou Patch applied
Alright, looking good to me :)
I'll wait for https://pagure.io/pagure-dist-git/pull-request/118 to get in and merge this one afterward :)
We can remove these two now ;-)
rebased onto f58df34fd54dc189cbd9003358bd2f4ca9e10fbc
rebased onto 75f7c960eaa0d8c615fbe0081737837691ccbb77
Tested locally and it works great, thanks for taking this! :)
Pull-Request has been merged by pingou
This PR adds option to add orphaning reason to package on dist-git. User
can choose from various options and write additional info if needed.
This is implementing the changes in Pagure needed for https://pagure.io/pagure-dist-git/pull-request/118.
I tried to run the pagure with dist-git on my machine, but without luck, so the changes here are untested :-(
Signed-off-by: Michal Konečný mkonecny@redhat.com