Skip to content

Conversation

selvaganesh0355
Copy link

  1. Enhanced explanations of webhook events and their significance.
  2. Clarification of the purpose and usage of each delivery header.
  3. Improved formatting and readability of the example webhook delivery.
  4. Added a "Testing Your Webhooks" section to simplify webhook testing for developers.
  5. Minor grammatical and stylistic improvements for better clarity and flow.

Copy link

welcome bot commented Apr 25, 2025

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Apr 25, 2025
Copy link
Contributor

How to review these changes 👓

Thank you for your contribution. To review these changes, choose one of the following options:

A Hubber will need to deploy your changes internally to review.

Table of review links

⚠️ Warning: Our review server is experiencing latency issues.

The table shows the files in the content directory that were changed in this pull request. This helps you review your changes on the review server. Changes to the data directory are not included in this table.

Source Review Production What Changed
webhooks/webhook-events-and-payloads.md fpt
ghec
ghes@ 3.16 3.15 3.14 3.13 3.12
fpt
ghec
ghes@ 3.16 3.15 3.14 3.13 3.12

Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server

🤖 This comment is automatically generated.

@Sharra-writes
Copy link
Contributor

Sharra-writes commented Apr 25, 2025

@selvaganesh0355 Thanks for opening a PR! Unfortunately, a lot of the changes you've made to this article don't conform to our style guide. Our contributing docs are a great place to start, too! For changes like the ones you're proposing here, we prefer that you begin by opening an issue, so that the necessity of the changes can be discussed.

Additionally, per our “About contributing to GitHub Docs” article -

We only document GitHub products, features, tools, and extensions. We may mention or link to third-party tools to demonstrate how a feature works, but we do not accept pull requests to document third-party tools or integrations unless they were codeveloped with GitHub.

Since most of these changes are just a reorganization and rewording of information already in the article, or link to third-party tools, I'm going to close this PR. If you would like to open an issue about the changes you would like to see made, though, we would be happy to discuss them in more detail!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage Do not begin working on this issue until triaged by the team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants