Skip to content

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

Merged
jaydgoss merged 1 commit into
masterfrom
jaygoss/fix-scm-platform-selected-auto-detect
Apr 20, 2026
Merged

fix(onboarding): Emit scm_platform_selected on auto-detection#113351
jaydgoss merged 1 commit into
masterfrom
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

linear-code Bot commented Apr 17, 2026

Copy link
Copy Markdown

@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
@jaydgoss
jaydgoss merged commit 85186ba into master Apr 20, 2026
65 checks passed
@jaydgoss
jaydgoss deleted the jaygoss/fix-scm-platform-selected-auto-detect branch April 20, 2026 16:32
@github-actions github-actions Bot locked and limited conversation to collaborators May 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants