Skip to content

fix: Drop events docs#14396

Merged
benjackwhite merged 5 commits intomasterfrom
claude/write-pr-docs-Kg0SQ
Jan 15, 2026
Merged

fix: Drop events docs#14396
benjackwhite merged 5 commits intomasterfrom
claude/write-pr-docs-Kg0SQ

Conversation

@benjackwhite
Copy link
Contributor

@benjackwhite benjackwhite commented Jan 15, 2026 •
edited
Loading

Changes

The Drop events docs showed twice and it was missing a bunch of standard context. Got Claude to clean it up

Checklist

  • Words are spelled using American English
  • PostHog product names are in title case. It's "Product Analytics" not "Product analytics". If talking about a category of product, use sentence case e.g. "There are a lot of product analytics tools, but PostHog's Product Analytics is the best"
  • Titles are in sentence case
  • Feature names are in sentence case. It's "Click here to create a trend insight" not "... create a Trend Insight" and so on.
  • Use relative URLs for internal links
  • If I moved a page, I added a redirect in vercel.json
  • Remove this template if you're not going to fill it out!

Article checklist

  • I've added (at least) 3-5 internal links to this new article
  • I've added keywords for this page to the rank tracker in Ahrefs
  • I've checked the preview build of the article
  • The date on the article is today's date
  • I've added this to the relevant "Tutorials and guides" docs page (if applicable)

@claude
Add documentation for secure event validation transformation
b688220 
Document the new secure event validation transformation from PR #42818.
Includes installation steps, client-side implementation examples,
secret rotation guide, and FAQs.
@vercel
Copy link

vercel bot commented Jan 15, 2026 •
edited
Loading

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

Project Deployment Review Updated (UTC)
posthog Ready Ready Preview Jan 15, 2026 0:30am

claude added 2 commits January 15, 2026 11:17
@claude
Fix drop-events doc formatting to match standard CDP template
6ec28d4 
- Add templateId to frontmatter
- Wrap Configuration/FAQ/snippets in HideOnCDPIndex
- Add standard PostHogMaintained and FeedbackQuestions snippets
- Remove showTitle that was causing sidebar issues
- Add Installation section for consistency
@claude
Add enforcing secure mode section with warning callout
acede88 
Emphasize that enforce mode should only be enabled after validating
that all events have correct verification hashes, as it will drop
any events without valid hashes.
@benjackwhite benjackwhite changed the title feat: Docs for secure event verification fix: Drop events docs Jan 15, 2026
@benjackwhite benjackwhite requested a review from a team January 15, 2026 12:11
@benjackwhite benjackwhite marked this pull request as ready for review January 15, 2026 12:11
meikelmosby
meikelmosby approved these changes Jan 15, 2026
View reviewed changes
Copy link
Contributor

@meikelmosby meikelmosby left a comment

Choose a reason for hiding this comment

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

checked it out and tested. lgtm

@benjackwhite benjackwhite enabled auto-merge (squash) January 15, 2026 12:28
@benjackwhite benjackwhite merged commit 2df326d into master Jan 15, 2026
15 checks passed
@benjackwhite benjackwhite deleted the claude/write-pr-docs-Kg0SQ branch January 15, 2026 12:30
sarahxsanders pushed a commit that referenced this pull request Jan 26, 2026
@benjackwhite @sarahxsanders
fix: Drop events docs (#14396)
59fdc20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@meikelmosby meikelmosby meikelmosby approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@benjackwhite @meikelmosby @claude