Skip to content

fix(onboarding): Emit scm_platform_selected on auto-detection#113351

Open
jaydgoss wants to merge 1 commit intomasterfrom
jaygoss/fix-scm-platform-selected-auto-detect
Open

fix(onboarding): Emit scm_platform_selected on auto-detection#113351
jaydgoss wants to merge 1 commit intomasterfrom
jaygoss/fix-scm-platform-selected-auto-detect

Conversation

@jaydgoss
Copy link
Copy Markdown
Member

Fire onboarding.scm_platform_selected with source: 'detected' once when the SCM onboarding platform auto-detection resolves and the user has not explicitly chosen a platform.

Previously the event only fired when the user clicked a detected platform card or picked manually. Users who accepted the recommended platform and advanced with Continue never emitted a platform-selected event, leaving the funnel without a step between scm_platform_features_step_viewed and scm_platform_features_continue_clicked.

The event is gated by a ref so it fires at most once per mount. A test asserts that exactly one source: 'detected' event fires when detection resolves on mount with no explicit selection.

Refs VDY-90

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 17, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 17, 2026
Fire `onboarding.scm_platform_selected` with `source: 'detected'` once
when platform auto-detection resolves and the user has not explicitly
chosen a platform. Previously the event only fired when the user
clicked on a detected card or picked manually, so users who accepted
the recommended platform and advanced with Continue never emitted a
platform-selected event and the funnel was missing a step.

Refs VDY-90
@jaydgoss jaydgoss force-pushed the jaygoss/fix-scm-platform-selected-auto-detect branch from 5283692 to c93621f Compare April 17, 2026 20:28
@jaydgoss jaydgoss marked this pull request as ready for review April 17, 2026 20:40
@jaydgoss jaydgoss requested a review from a team as a code owner April 17, 2026 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant