Skip to content

chore(auth): make Google One Tap the default, remove experiment flag#6326

Merged
capJavert merged 3 commits into
mainfrom
eng-1868-cleanup-google-one-tap-experiment-it-won
Jul 15, 2026
Merged

chore(auth): make Google One Tap the default, remove experiment flag#6326
capJavert merged 3 commits into
mainfrom
eng-1868-cleanup-google-one-tap-experiment-it-won

Conversation

@capJavert

@capJavert capJavert commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

What

The auth_google_onetap GrowthBook experiment won, so Google One Tap sign-in becomes the permanent baseline for all eligible users.

  • Removed the featureAuthGoogleOneTap flag definition from featureManagement.ts.
  • Dropped the useConditionalFeature flag gate in AuthOptionsInner.tsx; One Tap is now driven solely by the existing canUseOneTap eligibility guard (auth readiness, logged-out state, extension/iOS/Android-native exclusions).
  • Cleaned up the now-unused featureAuthGoogleOneTap and useConditionalFeature imports.

Key decisions

  • Per the flag-removal rule, kept the winning One Tap path permanently rather than making untested code always-on. The eligibility guard and all platform exclusions are untouched.
  • No changes to useGoogleOneTap, betterAuthOneTapCallback, or the analytics events — those are the winning behavior and stay as-is.

Testing

  • useGoogleOneTap.spec.ts (flag-agnostic, enabled-prop driven): 6/6 pass.
  • Lint clean on both impacted files.

Closes ENG-1868


Created by Huginn 🐦‍⬛

Preview domain

https://eng-1868-cleanup-google-one-tap.preview.app.daily.dev

capJavert and others added 2 commits July 15, 2026 09:02
The auth_google_onetap experiment won, so drop the flag gate and let
canUseOneTap alone drive One Tap. Removes featureAuthGoogleOneTap and
the now-unused useConditionalFeature wiring in AuthOptionsInner.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@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)
daily-webapp Ready Ready Preview Jul 15, 2026 12:48pm

Request Review

@capJavert capJavert disabled auto-merge July 15, 2026 12:44
@capJavert capJavert merged commit 36620cb into main Jul 15, 2026
3 of 5 checks passed
@capJavert capJavert deleted the eng-1868-cleanup-google-one-tap-experiment-it-won branch July 15, 2026 12:44
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