pagure

#2961 Fix editing and deleting comments added by the EV server to PRs
Merged by pingou. Opened by pingou.
fix_sse_prs  into  master

Download 2961.patch

Basically, we had the project url hard-coded in the html instead of have
it automatically generated from the content passed on by the EV server
so editing a comment added by the EV server was only ever going to work
on a one project.

In addition, the form and csrf token were missing to allow the delete
button to work.
Both form and csrf token are now present on the html page, thus allowing
to right away delete a generic comment that was added by the EV server.

Fixes https://pagure.io/pagure/issue/2958

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

Both commits can be reviewed independently.

1 new commit added

  • Move back to use then authenticated method

rebased onto e5c75b8c85df165db416e4f7953c7a344f7ab194

:thumbsup:

rebased onto ae92283730f6f242cb0e71dedd7a44b58ccaf566

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.