pagure

#2541 Fixes around the process to delete projects/forks
Merged by pingou. Opened by pingou.
del_forks  into  master

Download 2541.patch
no initial comment

1 new commit added

  • Fix running the tests and a variable name

This should probably be 403, since the route exists, it's just forbidden to delete projects on this instance of Pagure.

it considers these routes don't exists in these cases

@pingou just a minor comment about the HTTP status code used. Otherwise, +1.

rebased

I'm going to merge as is, thanks for the review and the suggestion, if you feel strongly about it let's discuss this in a ticket or a PR since this PR doesn't actually change the current behavior.

Thanks again

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.