Skip to content

docs(KNO-14363): document Amplitude source - #1564

Open
samseely wants to merge 2 commits into
mainfrom
sam/kno-14363-amplitude-source-docs
Open

docs(KNO-14363): document Amplitude source#1564
samseely wants to merge 2 commits into
mainfrom
sam/kno-14363-amplitude-source-docs

Conversation

@samseely

@samseely samseely commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an Amplitude source guide covering Webhooks Streaming and Cohort Webhooks
  • document shared-secret verification, default identify and audience mappings, identity alignment, custom workflow and group mappings, retries, idempotency, and debugging
  • add Amplitude to the sources sidebar and available-sources list

Validation

  • yarn format.check
  • yarn lint
  • yarn type-check
  • yarn build

Follow-up

  • add Amplitude-specific setup screenshots once an authenticated hosted Control preview is available

Linear: KNO-14363

@linear-code

linear-code Bot commented Jul 24, 2026

Copy link
Copy Markdown

KNO-14363

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jul 24, 2026 1:25am

Request Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk MEDIUM: Documents Amplitude as a new source integration with a new MDX page, sidebar entry, and overview listing.

Reasons

  • A new MDX page is added at content/integrations/sources/amplitude.mdx (248 lines), which triggers the new-page rule.
  • The sidebar navigation file data/sidebars/integrationsSidebar.ts is modified to add the Amplitude entry.
  • 250 total additions across 3 files is a moderately sized content addition.
  • The new page introduces several external Amplitude links and internal Knock doc links that should be verified for correctness.
  • No HIGH-risk triggers: no component, layout, lib, config, or dependency changes.

Notes

  • Verify that all external Amplitude documentation links resolve correctly (e.g., Webhooks Streaming, Cohort Webhooks, Identify API, retry mechanism, sync behaviors, FreeMaker templating, and user identity docs).
  • Verify that all internal Knock doc links resolve (/concepts/environments, /concepts/audiences, /concepts/objects, /concepts/tenants, /integrations/sources/overview#source-event-idempotency, /integrations/sources/overview#triggering-actions-from-source-events, /integrations/sources/custom).
  • Confirm the reused screenshot at /images/integrations/sources/sources-location.png is appropriate for the Amplitude setup context.
  • The sidebar entry is added in alphabetical order before Clerk — confirm this matches the intended navigation ordering.
  • Review the payload examples and action mapping tables for technical accuracy against the actual Amplitude webhook format and Knock source behavior.
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant