I'm trying to run the tests with tox, but so far I'm getting some gcc errors :(
rebased onto 5f7fc3845dfdeb5bc4b380073c167cf77366e5c5
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!
Thanks!
Pull-Request has been merged by pingou
I'm trying to run the tests with tox, but so far I'm getting some gcc errors :(