Skip to content

Conversation

@ssganesh035
Copy link
Contributor

  1. Provided step-by-step instructions for using both tools.
  2. Clarified webhook security best practices, including token-based authentication.
  3. Improved formatting and readability for better developer experience.
@ssganesh035 ssganesh035 requested a review from pwizla as a code owner February 21, 2025 16:41
@strapi-cla
Copy link

strapi-cla commented Feb 21, 2025

CLA assistant check
All committers have signed the CLA.

@vercel
Copy link

vercel bot commented Feb 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 26, 2025 6:38pm
Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution, @ssganesh035!

I'll do a deeper review soon, but for now I can already request 2 types of changes:

  • Please don't use capitalized letters on heading titles. Available ConfigurationsAvailable configurations
  • Please don't delete content that we commented out. We keep it for further reference as this might be reintroduced 🙏
@pwizla pwizla self-assigned this Feb 24, 2025
@pwizla pwizla added source: Dev Docs PRs/issues targeting the Developer Docs pr: updated content PRs updating existing documentation content temp - port to docs-next (Temporary label) The content of this PR should be ported to docs-next.strapi.io labels Feb 24, 2025
@pwizla pwizla added this to the v5.6.3 milestone Feb 24, 2025
@ssganesh035
Copy link
Contributor Author

Hi @pwizla, I have made the suggested changes. Let me know if you want any other changes.

- Use secure, HTTPS endpoints for receiving webhooks.
- Set up rate limiting to avoid being overwhelmed by multiple webhook requests.

## Webhooks testing
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this section. Strapi Docs are not intended to promote 3rd party proprietary tools.

@pwizla
Copy link
Collaborator

pwizla commented Feb 25, 2025

Thank you for the updates, @ssganesh035. You just need to remove the last section and your contribution will be merged. 🙂

@ssganesh035
Copy link
Contributor Author

I have made the changes, I hope it's good to go.

Copy link
Collaborator

@pwizla pwizla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now good to go! Thank you :)

@pwizla pwizla changed the title Enhanced Strapi Webhooks Docs Enhanced webhooks docs Feb 26, 2025
@pwizla pwizla merged commit 2a5d617 into strapi:main Feb 26, 2025
3 checks passed
@pwizla pwizla removed the temp - port to docs-next (Temporary label) The content of this PR should be ported to docs-next.strapi.io label Mar 20, 2025
pwizla added a commit that referenced this pull request Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: updated content PRs updating existing documentation content source: Dev Docs PRs/issues targeting the Developer Docs

3 participants