Skip to content

Commit e7e37ef

Browse files
committed
fix(onboarding): gate profile auto complete behind v2
1 parent 81433e3 commit e7e37ef

1 file changed

Lines changed: 13 additions & 4 deletions

File tree

packages/shared/src/components/auth/AuthOptionsInner.tsx

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,8 @@ import {
5959
DATE_SINCE_ACTIONS_REQUIRED,
6060
onboardingCompletedActions,
6161
} from '../../hooks/auth';
62+
import { useConditionalFeature } from '../../hooks/useConditionalFeature';
63+
import { featureOnboardingV2 } from '../../lib/featureManagement';
6264

6365
const AuthDefault = dynamic(
6466
() => import(/* webpackChunkName: "authDefault" */ './AuthDefault'),
@@ -177,6 +179,12 @@ function AuthOptionsInner({
177179
const router = useRouter();
178180
const isOnboardingOrFunnel =
179181
!!router?.pathname?.startsWith('/onboarding') || isFunnel;
182+
const { value: isOnboardingV2 } = useConditionalFeature({
183+
feature: featureOnboardingV2,
184+
shouldEvaluate: trigger === AuthTriggers.Onboarding,
185+
});
186+
const shouldAutoCompleteOnboarding =
187+
trigger === AuthTriggers.Onboarding && isOnboardingV2;
180188
const [activeDisplay, setActiveDisplay] = useState(() =>
181189
storage.getItem(SIGNIN_METHOD_KEY) && !forceDefaultDisplay
182190
? AuthDisplay.SignBack
@@ -372,7 +380,7 @@ function AuthOptionsInner({
372380
}
373381
} else if (trigger === AuthTriggers.RecruiterSelfServe) {
374382
await autoCompleteProfile(user.email, user.name, false);
375-
} else if (trigger === AuthTriggers.Onboarding) {
383+
} else if (shouldAutoCompleteOnboarding) {
376384
await autoCompleteProfile(user.email, user.name, acceptedMarketing);
377385
} else {
378386
onSetActiveDisplay(AuthDisplay.SocialRegistration);
@@ -484,12 +492,13 @@ function AuthOptionsInner({
484492

485493
if (
486494
trigger === AuthTriggers.RecruiterSelfServe ||
487-
trigger === AuthTriggers.Onboarding
495+
shouldAutoCompleteOnboarding
488496
) {
489497
setIsSocialAuthLoading(false);
490498
const loggedUser = boot.user as LoggedUser;
491-
const marketing =
492-
trigger === AuthTriggers.Onboarding ? acceptedMarketing : false;
499+
const marketing = shouldAutoCompleteOnboarding
500+
? acceptedMarketing
501+
: false;
493502
await autoCompleteProfile(loggedUser.email, loggedUser.name, marketing);
494503
return;
495504
}

0 commit comments

Comments
 (0)