#4946 Add orphaning reason
Merged by pingou. Opened by zlopez.
zlopez/pagure orphan  into  master

Download 4946.patch

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:

dropdown.png

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

1 new commit added

  • 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

:thumbsup:

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

0001-Rework-the-UI-UX-around-orphaning-adopting-orphaned-.patch

Here are a couple of screenshots:
Screenshot_from_2020-08-19_16-43-55.png
Screenshot_from_2020-08-19_16-44-00.png

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

Metadata