Skip to content

docs: address review feedback on experimentation guides#8016

Open
Zaimwa9 wants to merge 1 commit into
mainfrom
fix/docs-experimentation-review-feedback
Open

docs: address review feedback on experimentation guides#8016
Zaimwa9 wants to merge 1 commit into
mainfrom
fix/docs-experimentation-review-feedback

Conversation

@Zaimwa9

@Zaimwa9 Zaimwa9 commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Changes

Follow-up to #7993 — addresses post-merge review from @Holmus.

  • Rename "Experimentation (A/B Testing)" → "A/B Testing with Integrations" to disambiguate sidebar
  • Add no-cost reassurance, support link, processing-time admonition to warehouse page
  • Add SDK availability note and self-managed reassurance to index
  • Clarify "cannot be undone", case sensitivity, SRM warning
  • Add create-metric screenshot
  • Fix broken anchor, double "manually", awkward wording

How did you test this code?

npm run build in docs/ — no broken links, prettier clean.

@vercel

vercel Bot commented Jul 15, 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 15, 2026 11:50am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
flagsmith-frontend-preview Ignored Ignored Jul 15, 2026 11:50am
flagsmith-frontend-staging Ignored Ignored Jul 15, 2026 11:50am

Request Review

@coderabbitai

coderabbitai Bot commented Jul 15, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 71b10ba8-089a-4a85-85e1-8b0e94cc1d54

📥 Commits

Reviewing files that changed from the base of the PR and between 9e4b1f4 and a4bb22c.

⛔ Files ignored due to path filters (1)
  • docs/static/img/experimentation/create-metric-form.png is excluded by !**/*.png
📒 Files selected for processing (7)
  • docs/docs/experimentation-ab-testing.md
  • docs/docs/experimentation/analyse-an-experiment.md
  • docs/docs/experimentation/connect-a-warehouse.md
  • docs/docs/experimentation/create-an-experiment.md
  • docs/docs/experimentation/create-metrics.md
  • docs/docs/experimentation/index.md
  • docs/docs/experimentation/run-an-experiment.mdx

📝 Walkthrough

Walkthrough

Updated experimentation documentation to clarify page naming, environment scope, glossary and SDK availability, metric creation, integrations, warehouse connections, experiment ending, event collection, conversion-event handling, and sample ratio mismatch guidance.

Estimated code review effort: 2 (Simple) | ~10 minutes


Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added the docs Documentation updates label Jul 15, 2026
@Zaimwa9 Zaimwa9 marked this pull request as ready for review July 15, 2026 15:28
@Zaimwa9 Zaimwa9 requested a review from a team as a code owner July 15, 2026 15:28
@Zaimwa9 Zaimwa9 requested review from Holmus and adamvialpando and removed request for a team and adamvialpando July 15, 2026 15:28
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.

1 participant