Skip to content

Conversation

@TyroneOrrego
Copy link
Contributor

@TyroneOrrego TyroneOrrego commented Jan 9, 2026

Thanks for submitting a PR! Please check the boxes below:

  • [X ] I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

Contributes to #6472

This PR fixes the webhook documentation that had been mixed together. It restores the Environment Webhooks section in Environment Settings (previously missing), removes the Audit Log Webhooks section from the Identity webhooks page, and clarifies the Identity webhooks use case for analytics processing, cohort analysis, and A/B testing. It also adds the complete payload schema to the Audit Log webhooks page and clearly separates webhook configuration from audit log event types, so readers can tell what applies to webhooks versus the audit log itself. The three webhook types (Identity, Environment, and Audit Log) are now documented separately with their own use cases, payloads, and setup instructions.

How did you test this code?

Please describe.

@TyroneOrrego TyroneOrrego requested a review from a team as a code owner January 9, 2026 16:02
@TyroneOrrego TyroneOrrego requested review from matthewelwell and removed request for a team January 9, 2026 16:02
@vercel
Copy link

vercel bot commented Jan 9, 2026

@TyroneOrrego is attempting to deploy a commit to the Flagsmith Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the docs Documentation updates label Jan 9, 2026
@TyroneOrrego TyroneOrrego changed the title Webhook docs: enhance webhook documentation for audit logs and environment settings Jan 9, 2026
@vercel
Copy link

vercel bot commented Jan 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
docs Ready Ready Preview, Comment Jan 19, 2026 9:29am
flagsmith-frontend-preview Ready Ready Preview, Comment Jan 19, 2026 9:29am
flagsmith-frontend-staging Ready Ready Preview, Comment Jan 19, 2026 9:29am

Request Review

Holmus
Holmus previously approved these changes Jan 19, 2026
Copy link
Contributor

@Holmus Holmus left a comment

Choose a reason for hiding this comment

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

Two minor nitpicks, LGTM though!

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

Labels

docs Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: webhook documentation missing / unclear

2 participants