Skip to content
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

Webhook for internal wiki #16457

Closed
SohnyBohny opened this issue Jul 16, 2021 · 1 comment · Fixed by #20219
Closed

Webhook for internal wiki #16457

SohnyBohny opened this issue Jul 16, 2021 · 1 comment · Fixed by #20219
Labels
topic/webhooks type/feature Completely new functionality. Can only be merged if feature freeze is not active.

Comments

@SohnyBohny
Copy link
Contributor

As far as I see there are currently no Webhooks for the internal wiki.
I think triggering a web hook on a wiki change/commit would be a nice feature.

@noerw noerw added topic/webhooks type/feature Completely new functionality. Can only be merged if feature freeze is not active. labels Jul 16, 2021
@f0086
Copy link
Contributor

f0086 commented Aug 2, 2022

I've made a PR for this to support webhooks for the internal wiki a month ago. Still waiting for a proper review.

zeripath pushed a commit that referenced this issue Sep 4, 2022
Add support for triggering webhook notifications on wiki changes.

This PR contains frontend and backend for webhook notifications on wiki actions (create a new page, rename a page, edit a page and delete a page). The frontend got a new checkbox under the Custom Event -> Repository Events section. There is only one checkbox for create/edit/rename/delete actions, because it makes no sense to separate it and others like releases or packages follow the same schema.

![image](https://user-images.githubusercontent.com/121972/177018803-26851196-831f-4fde-9a4c-9e639b0e0d6b.png)

The actions itself are separated, so that different notifications will be executed (with the "action" field). All the webhook receivers implement the new interface method (Wiki) and the corresponding tests.

When implementing this, I encounter a little bug on editing a wiki page. Creating and editing a wiki page is technically the same action and will be handled by the ```updateWikiPage``` function. But the function need to know if it is a new wiki page or just a change. This distinction is done by the ```action``` parameter, but this will not be sent by the frontend (on form submit). This PR will fix this by adding the ```action``` parameter with the values ```_new``` or ```_edit```, which will be used by the ```updateWikiPage``` function.

I've done integration tests with matrix and gitea (http).

![image](https://user-images.githubusercontent.com/121972/177018795-eb5cdc01-9ba3-483e-a6b7-ed0e313a71fb.png)

Fix #16457

Signed-off-by: Aaron Fischer <mail@aaron-fischer.net>
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
topic/webhooks type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants