You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When I used the actual webhook, I saw that there is still some "Gogs" implementation, so first we should modify it.
I would also suggest to modify the webhook implementation by dividing it into two sort of webhook.
One for incoming and another for outgoing (as it is right now).
If we implement incoming webhook, it can be a nice implementation for CI result of commit or PR (see #996).
For the implementation on the UI, I would suggest either add another button for "Incoming Webhook" and rename "Add Webhook" to "Outgoing Webhook" or use icons in the list like the following ones:
What do you think about those improvements?
The text was updated successfully, but these errors were encountered:
For Outgoing webhook, it's only when Gitea will POST to another url like Rocket.Chat, Mattermost,...
The Incoming webhooks is when another application POST to Gitea that will handle the post to do something.
I can see a use of incoming webhook to drone for example POST a result and Gitea will handle it to update a table with the build status.
After digging to the code and on github api, I found this (https://developer.github.com/v3/repos/statuses/).
My proposal for incoming webhook does not have to exist because of the API that I forgot :p.
So the only thing I think we should do is to remove the "GOGS" information in all webhook.
See, there for example.
When I used the actual webhook, I saw that there is still some "Gogs" implementation, so first we should modify it.
I would also suggest to modify the webhook implementation by dividing it into two sort of webhook.
One for incoming and another for outgoing (as it is right now).
If we implement incoming webhook, it can be a nice implementation for CI result of commit or PR (see #996).
For the implementation on the UI, I would suggest either add another button for "Incoming Webhook" and rename "Add Webhook" to "Outgoing Webhook" or use icons in the list like the following ones:
What do you think about those improvements?
The text was updated successfully, but these errors were encountered: