#4771 Use the package_unretirement template with releng unretirement
Merged by pingou. Opened by churchyard.
churchyard/pagure unretire_template  into  master

Download 4771.patch

I'm trying to run the tests with tox, but so far I'm getting some gcc errors :(

rebased onto 5f7fc3845dfdeb5bc4b380073c167cf77366e5c5

I'm trying to run the tests with tox, but so far I'm getting some gcc errors :(

I give up. If this template is tested, please let me know where and I'll adapt the test blindly.

The src.fp.o theme isn't tested, but I'll test it locally quickly :)

Thanks.

To get it to work I also had to add:

             _btn.prop( "title", "Package retired - Open a releng ticket to adopt it" );
             _btn.html("Retired");
+            _btn.removeClass('disabled');

Could you add it to your PR?

Or simpler:

             });
             _btn.prop( "title", "Package retired - Open a releng ticket to adopt it" );
             _btn.html("Retired");
-          } else {
-            _btn.removeClass('disabled');
           }
+          _btn.removeClass('disabled');
         }
       });

I can but I don't understand why. I only want to change the link.

The button remains disabled for me when I test this locally, meaning clicking on it doesn't send to the releng project. I can put this in a different PR if you prefer

Like that?

1 new commit added

  • Move "disabled" code around as @pingou told me to

The code and change looks good, the commit message a little less :(

I can put a real rationale in there, but I really have no idea what it is.

If you tell me the commit message, I'll amend.

How does this reads?

When the package is orphaned, the "take" button shows as disabled in the UI, until we check if the package is orphaned and retired or just orphaned. Except that in the case where the package is orphaned and retired, we re-purpose that button to send the users to the releng issue tracker to ask for the package to be un-retired/un-blocked, but we did not re-enable the button. This led to a button being re-purposed but not re-actived, and thus clicking on the button did not have any effect.
This commit fixes that by making sure that we always re-enable the button, either because the package is orphaned and not retired and packagers can just adopt it, or because the package is orphaned and retired and by clicking on the button packagers will be sent to the releng issue tracker where they'll be able to ask for the package to be un-blocked.

2 new commits added

  • Make sure orphaned & retired packages can be claimed
  • Use the package_unretirement template with releng unretirement

Perfect!

Pull-Request has been merged by pingou

Metadata