-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Gitea sends discord webhook for edited release draft #27157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
lunny
pushed a commit
that referenced
this issue
Feb 2, 2024
GiteaBot
pushed a commit
to GiteaBot/gitea
that referenced
this issue
Feb 2, 2024
silverwind
pushed a commit
that referenced
this issue
Feb 2, 2024
…29025) Backport #29008 by @yp05327 Fix #27157 Co-authored-by: yp05327 <[email protected]>
silverwind
pushed a commit
to silverwind/gitea
that referenced
this issue
Feb 20, 2024
Automatically locked because of our CONTRIBUTING guidelines |
project-mirrors-bot-tu bot
pushed a commit
to project-mirrors/forgejo-as-gitea-fork
that referenced
this issue
Jan 23, 2025
…#29008) (go-gitea#29025) Backport go-gitea#29008 by @yp05327 Fix go-gitea#27157 Co-authored-by: yp05327 <[email protected]> (cherry picked from commit 8def405)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
When you set up a discord-type webhook where you chose your events and selected
Release
will Gitea not send a Webhook when you save a new release as draft. This is fine and something I would expect as you may not want to publish new info too early to everyone.However, if you edit said draft and again save it as such, therefore editing the draft, will Gitea send a webhook with the edit notification, which isn't really an optimal thing to do here.
Note that I have encountered this behaviour on a forgejo instance and I cannot test it within try.gitea.io due to it not allowing to make release drafts... So testing it yourself if you have the permission - or granting me the permission (Username is Andre601) to make drafts - is recommended.
Steps for possible reproduction:
I would like to take the oportunity to also propose changes to the webhook system.
The way it is right now is flexible, yet rigit.
Flexible, because you can define your own selection of events to send a webhook for, but rigit because the selectable events group common types together, without any way of filtering them out...
As an example "Release" includes creation, deletion and edits of releases, without having any option to only have creation and edits f.e.
It should be considered to allow more fine-grained selections. Maybe by having the types as sub-lists of the event itself to select, where selecting the main/parent event would auto-select all the sub-list items...
A quick mockup using GitHub checkbox lists:
I can create a separate issue for this as a feature request if that is wanted.
Gitea Version
1.20.3+0-38-g0e33ecb13
Can you reproduce the bug on the Gitea demo site?
No (Not possible to test on that instance)
Log Gist
No response
Screenshots
Webhook message of the edited release draft:

The release in question, clearly marked as Draft:

Git Version
Operating System
How are you running Gitea?
I do not self-host/run it, but use codeberg.org for this.
As mentioned is the instance actually using forgejo, but given its soft-fork nature and currently not having any major differences from Gitea, I have strong beliefs that this bug exists within Gitea itself.
Database
None
The text was updated successfully, but these errors were encountered: