feat: signup corporate-only experiment with exposure and new_signup events#7979
feat: signup corporate-only experiment with exposure and new_signup events#7979Zaimwa9 wants to merge 8 commits into
Conversation
This reverts commit 56c672e.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 WalkthroughWalkthroughThe signup flow now records Flagsmith Estimated code review effort: 3 (Moderate) | ~20 minutes Comment |
Docker builds report
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
|
Visual Regression19 screenshots compared. See report for details. |
There was a problem hiding this comment.
Actionable comments posted: 5
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: a04fcc89-95a6-4ac2-aec1-a2a6a17d5749
⛔ Files ignored due to path filters (1)
frontend/package-lock.jsonis excluded by!**/package-lock.json
📒 Files selected for processing (4)
frontend/common/stores/account-store.jsfrontend/common/useSignupExperiment.tsfrontend/package.jsonfrontend/web/components/pages/HomePage.tsx
Thanks for submitting a PR! Please check the boxes below:
docs/if required so people know about the feature.Changes
Sets up our first dogfood experiment, Signup corporate domains only.
The experiment tests whether requiring a work email address at signup affects conversion. Control keeps today's behaviour (an informational message for free email domains); the
signup_corporate_onlyvariant disables the Create Account button for free email domains and shows "Please use your work email address to create your account."@flagsmith/flagsmithto 12.1.0 fortrackEvent,trackExposureEvent,getExperimentFlagand thevariantkey on flags.useSignupExperimenthook: identifies anonymous signup visitors with a transient identity (persistentsignup_anonymous_idso returning visitors keep their variant), resolves thesignup_corporate_onlyflag's variant, and records a$flag_exposure. Takes a boolean to fall back to environment flags — no identify, no exposure, control experience — for non-participants (invitees, login views, logged-in users,preventEmailPassword).new_signupconversion event on successful registration, attributed to the same anonymous identity, withfree_email_domain,invite,signup_methodandutm_sourcemetadata.How did you test this code?
Manually, against the staging configuration with the flag and experiment set up:
/signuplogged out — the network tab showsidentities/?identifier=<uuid>&transient=true, followed (≤10s) by aPOST /v1/eventscontaining a$flag_exposurewith the variant asvalue(verified 202).new_signupwith the same identifier and metadata.