ogr icon indicating copy to clipboard operation
ogr copied to clipboard

WIP: Add and implement PullRequest.reopen()

Open jpopelka opened this issue 2 years ago • 14 comments

I'd like to use this functionality in hardly.

http POST https://pagure.io/api/0/ogr-tests/pull-request/16/close 'Authorization:token my-token' still returns "Invalid or expired token..." even I just generated it with all ACLs. In case it works for you, please uncomment the test_pr_close_reopen() for pagure and generate the response files. Thanks!!

RELEASE NOTES BEGIN PullRequest now contains reopen() method implemented by its subclasses. RELEASE NOTES END

jpopelka avatar Apr 29 '22 15:04 jpopelka

Build failed.

recheck

jpopelka avatar Apr 29 '22 15:04 jpopelka

Build failed.

I need to look at the Pagure again since the /reopen endpoint doesn't seem to exist in pagure.io or src.fedoraproject.org even it should have been implemented since https://pagure.io/pagure/pull-request/3349

jpopelka avatar Apr 29 '22 16:04 jpopelka

Pagure API support here: https://pagure.io/pagure/pull-request/5291

mfocko avatar May 01 '22 12:05 mfocko

You're my personal hero, Maťo!

jpopelka avatar May 02 '22 07:05 jpopelka

This issue has been marked as stale because it hasn't seen any activity for the last 60 days.

Stale issues are closed after 14 days, unless the label is removed by a maintainer or someone comments on it.

This is done in order to ensure that open issues are still relevant.

Thank you for your contribution! :unicorn: :rocket: :robot:

(Note: issues labeled with pinned or EPIC are never marked as stale.)

stale[bot] avatar Jul 10 '22 07:07 stale[bot]

pinged Pagure side... once again...

mfocko avatar Jul 11 '22 09:07 mfocko

Unblocked, not yet deployed

mfocko avatar Aug 19 '22 05:08 mfocko

This issue has been marked as stale because it hasn't seen any activity for the last 60 days.

Stale issues are closed after 14 days, unless the label is removed by a maintainer or someone comments on it.

This is done in order to ensure that open issues are still relevant.

Thank you for your contribution! :unicorn: :rocket: :robot:

(Note: issues labeled with pinned or EPIC are never marked as stale.)

stale[bot] avatar Oct 30 '22 13:10 stale[bot]

Merged on Pagure' side, not yet released neither deployed…

mfocko avatar Oct 30 '22 20:10 mfocko

This issue has been marked as stale because it hasn't seen any activity for the last 60 days.

Stale issues are closed after 14 days, unless the label is removed by a maintainer or someone comments on it.

This is done in order to ensure that open issues are still relevant.

Thank you for your contribution! :unicorn: :rocket: :robot:

(Note: issues labeled with pinned or EPIC are never marked as stale.)

stale[bot] avatar Jan 07 '23 23:01 stale[bot]

Still waiting for >5.13.3 release

jpopelka avatar Jan 09 '23 11:01 jpopelka

I'm considering finishing this with a local deployment, cause I have a feeling this PR will live longer than all of us…

mfocko avatar Jan 13 '23 11:01 mfocko