fix(onboarding): let segment overrides match the entry decision (#8288)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: talissoncosta

frontend/common/utils/onboardingEntry.ts

@@ -20,9 +20,13 @@ export type OnboardingEntryDecision = {
  * the logged-in user, and the routing it should have driven has happened.
  * Call `persistOnboardingEntry` with an accepted decision.
  */
-export async function decideOnboardingEntry(): Promise<OnboardingEntryDecision> {
+export async function decideOnboardingEntry(
+  email?: string,
+): Promise<OnboardingEntryDecision> {
+  // Only used to match segment overrides, not for bucketing. An override on
+  // another trait needs that trait passed here too.
   // @ts-expect-error transient is missing from the SDK's identify type
-  await flagsmith.identify('', {}, true)
+  await flagsmith.identify('', email ? { email } : {}, true)
   const flag = flagsmith.getExperimentFlag('onboarding_quickstart_flow')
   const identifier = flagsmith.getContext().identity?.identifier
   const variant: OnboardingVariant =

frontend/web/components/App.js

@@ -163,7 +163,9 @@ const App = class extends Component {
       // instead of blocking the redirect.
       Promise.race([
         AccountStore.getUser()?.isGettingStarted
-          ? decideOnboardingEntry().catch(() => null)
+          ? decideOnboardingEntry(AccountStore.getUser()?.email).catch(
+              () => null,
+            )
           : Promise.resolve(null),
         new Promise((resolve) => setTimeout(() => resolve(null), 2000)),
       ]).then((decision) => {