Webhook for removal of stale reviews and dismissing of reviews #10261

Open
opened 2025-11-02 09:02:32 -06:00 by GiteaMirror · 3 comments
Owner

Originally created by @gempir on GitHub (Feb 13, 2023).

Feature Description

There are 2 ways that I know of how a review can get removed from a pull request.

Either through the branch protection feature of removing stale reviews or dismissing the review in the UI.

Neither of them trigger a webhook. It would be nice to have a webhook to handle these cases.

My example usecase is syncing the approval state to a message in slack (via reactions)
Sadly currently I can't remove the reaction when an approval is dismissed or stale.

It should probably part of the "Pull Request Reviewed" PR events in the webhooks UI.

Screenshots

Stale
image

Dismiss
image

Edit Webhook Events
image

Originally created by @gempir on GitHub (Feb 13, 2023). ### Feature Description There are 2 ways that I know of how a review can get removed from a pull request. Either through the branch protection feature of removing stale reviews or dismissing the review in the UI. Neither of them trigger a webhook. It would be nice to have a webhook to handle these cases. My example usecase is syncing the approval state to a message in slack (via reactions) Sadly currently I can't remove the reaction when an approval is dismissed or stale. It should probably part of the "Pull Request Reviewed" PR events in the webhooks UI. ### Screenshots Stale ![image](https://user-images.githubusercontent.com/9765622/218406229-33a5d869-c12e-4b61-bcfe-a64f88f306aa.png) Dismiss ![image](https://user-images.githubusercontent.com/9765622/218406397-ffbbae5f-557d-42d6-aeb3-c6de326b4c69.png) Edit Webhook Events ![image](https://user-images.githubusercontent.com/9765622/218407072-4601c9e5-f37b-431d-8656-b65c14926f7f.png)
GiteaMirror added the type/proposaltype/feature labels 2025-11-02 09:02:32 -06:00
Author
Owner

@Kafumio commented on GitHub (Jun 3, 2023):

hi @lunny , I am interested in this issue and would like to try implementing this feature. Can you assign it to me?

@Kafumio commented on GitHub (Jun 3, 2023): hi @lunny , I am interested in this issue and would like to try implementing this feature. Can you assign it to me?
Author
Owner

@gempir commented on GitHub (Jun 3, 2023):

The odds of someone implementing it at the same time are very low. Why do you need to be assigned. Just do it.

@gempir commented on GitHub (Jun 3, 2023): The odds of someone implementing it at the same time are very low. Why do you need to be assigned. Just do it.
Author
Owner

@Kafumio commented on GitHub (Jun 3, 2023):

ok, I will do it

@Kafumio commented on GitHub (Jun 3, 2023): ok, I will do it
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#10261