feat: add workflow_run:completed webhook, set empty client_payload for workflow_dispatch webhook
#139
Conversation
heuels
changed the title
client_payload for workflow_dispatch webhookworkflow_run:completed webhook, set empty client_payload for workflow_dispatch webhook
| @@ -20593,16 +20593,13 @@ | |||
| }, | |||
| { | |||
| "name": "repository_dispatch", | |||
| "description": "This event occurs when a GitHub App sends a `POST` request to the \"[Create a repository dispatch event](https://docs.github.com/en/v3/repos/#create-a-repository-dispatch-event)\" endpoint.\n\nGitHub Apps must have the `contents` permission to receive this webhook.", | |||
| "description": "This event occurs when a GitHub App sends a `POST` request to the \"[Create a repository dispatch event](https://docs.github.com/en/v3/repos/#create-a-repository-dispatch-event)\" endpoint.", | |||
There was a problem hiding this comment.
the description is directly parsed from the website, this change will be reverted with next next automated update
There was a problem hiding this comment.
I haven't edited index.json manually – I ran bin/octokit-webhooks.js update and that's what was generated.
There was a problem hiding this comment.
As far as I can see, they've recently moved this note to a separate section: https://docs.github.com/en/developers/webhooks-and-events/webhook-events-and-payloads#availability-35.
|
@gr2m, the schedule section of the webhooks/.github/workflows/update.yml Lines 2 to 4 in b4ece11 |
|
oopsies, that should have been uncommented as part of #134. When you ran |
|
Done 🙂 |
|
🎉 This PR is included in version 3.12.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Hi!
Just a minor thing, but it still looks worth fixing while I battle the JSON Schema :)