Deleting comment in review triggers dicord webhook #14689

Closed
opened 2025-11-02 11:20:10 -06:00 by GiteaMirror · 1 comment
Owner

Originally created by @TheFox0x7 on GitHub (Jul 1, 2025).

Description

Noticed on accident while reviewing fixes for tea.
I've deleted a comment from review before posting the review but the notification about comment being deleted showed up in discord logs, which I don't think is correct behavior.

Steps to reproduce:

  1. Setup webhook with delete notifications (not sure about exact config here)
  2. Start a review and add a comment
  3. Delete the comment from the review

Gitea Version

1.25.0+dev-207-ge3887f984e

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

no access

Screenshots

Image

Git Version

no access

Operating System

no access

How are you running Gitea?

Reproduced on accident with gitea.com deployment

Database

None

Originally created by @TheFox0x7 on GitHub (Jul 1, 2025). ### Description Noticed on accident while reviewing fixes for tea. I've deleted a comment from review before posting the review but the notification about comment being deleted showed up in discord logs, which I don't think is correct behavior. Steps to reproduce: 1. Setup webhook with delete notifications (not sure about exact config here) 2. Start a review and add a comment 3. Delete the comment from the review ### Gitea Version 1.25.0+dev-207-ge3887f984e ### Can you reproduce the bug on the Gitea demo site? Yes ### Log Gist no access ### Screenshots ![Image](https://github.com/user-attachments/assets/ee603389-3dfe-4297-b935-584d724849d4) ### Git Version no access ### Operating System no access ### How are you running Gitea? Reproduced on accident with gitea.com deployment ### Database None
GiteaMirror added the type/bugissue/duplicate labels 2025-11-02 11:20:10 -06:00
Author
Owner

@delvh commented on GitHub (Jul 1, 2025):

Duplicate of #18846, I think that it's the same root problem

@delvh commented on GitHub (Jul 1, 2025): Duplicate of #18846, I think that it's the same root problem
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#14689