Skip to content

fix(analytics): Preserve setup-docs project attribution - #120229

Merged
jaydgoss merged 1 commit into
masterfrom
jaygoss/scm-project-creation-attribution-payloads
Jul 24, 2026
Merged

fix(analytics): Preserve setup-docs project attribution#120229
jaydgoss merged 1 commit into
masterfrom
jaygoss/scm-project-creation-attribution-payloads

Conversation

@jaydgoss

Copy link
Copy Markdown
Member

TLDR

Project-creation setup-docs events now retain the project and platform identifiers BI needs to join downstream activity to the project that was created.

Details

The project-creation BI audit found two payload-shape gaps after the getting-started handoff. project_creation.dsn_copied carried platform and variant but not project_id, while the getting-started removal events populated their platform field with the project slug.

A small docsFlowProjectIdParams helper now adds project_id to DSN-copy events for the project-creation and unmarked project surfaces. The onboarding and SCM-onboarding arms keep their existing payload shape, matching the established rule that their distinct event names remain unchanged.

PlatformDocHeader now reports platform.id for data-removal confirmation and completion instead of recentCreatedProject.slug. The callback dependency and regression expectations move with that value so the event contract consistently identifies the SDK platform rather than the project.

Focused coverage locks the four docs-flow project-ID cases, the unmarked fallback, both onboarding exclusions, and the corrected platform payload for marked and unmarked getting-started views.

Refs VDY-133

@linear-code

linear-code Bot commented Jul 21, 2026

Copy link
Copy Markdown

VDY-133

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 21, 2026
@jaydgoss
jaydgoss marked this pull request as ready for review July 21, 2026 17:47
@jaydgoss
jaydgoss requested review from a team as code owners July 21, 2026 17:47
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-platform-analytics-completeness branch from 0324dd4 to d0a5c6b Compare July 21, 2026 17:53
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-attribution-payloads branch from 2704d0d to c73a1a8 Compare July 21, 2026 17:57
@jaydgoss
jaydgoss changed the base branch from jaygoss/scm-project-creation-platform-analytics-completeness to jaygoss/scm-project-creation-framework-action-variants July 21, 2026 17:57
@jaydgoss
jaydgoss marked this pull request as draft July 21, 2026 18:01
@jaydgoss
jaydgoss marked this pull request as ready for review July 21, 2026 18:06
@bruno-garcia

Copy link
Copy Markdown
Member

@sentry review

@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-framework-action-variants branch from 1f03a96 to fa84769 Compare July 23, 2026 19:24
@jaydgoss
jaydgoss requested review from a team as code owners July 23, 2026 19:24
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-attribution-payloads branch from c73a1a8 to f2ad681 Compare July 23, 2026 19:24
@sentry

sentry Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 451 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-framework-action-variants branch from fa84769 to 8144860 Compare July 23, 2026 20:42
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-attribution-payloads branch from f2ad681 to 249d4a0 Compare July 23, 2026 20:42
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-framework-action-variants branch from 8144860 to 1004f0d Compare July 23, 2026 20:54
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-attribution-payloads branch from 249d4a0 to 8f6b657 Compare July 23, 2026 20:54
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-framework-action-variants branch from 1004f0d to 1a51c2b Compare July 23, 2026 21:09
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-attribution-payloads branch from 8f6b657 to d8324d5 Compare July 23, 2026 21:09
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-framework-action-variants branch from 1a51c2b to 7a152c6 Compare July 24, 2026 01:41
Base automatically changed from jaygoss/scm-project-creation-framework-action-variants to master July 24, 2026 01:52
Add project_id to project-creation DSN copy events while preserving onboarding payloads. Report the platform ID instead of the project slug in getting-started removal events.

Refs VDY-133
@jaydgoss
jaydgoss force-pushed the jaygoss/scm-project-creation-attribution-payloads branch from d8324d5 to 5b14077 Compare July 24, 2026 01:53
@jaydgoss
jaydgoss merged commit 0dc97a2 into master Jul 24, 2026
77 of 78 checks passed
@jaydgoss
jaydgoss deleted the jaygoss/scm-project-creation-attribution-payloads branch July 24, 2026 02:09
Christinarlong pushed a commit that referenced this pull request Jul 24, 2026
## TLDR

Project-creation setup-docs events now retain the project and platform
identifiers BI needs to join downstream activity to the project that was
created.

## Details

The project-creation BI audit found two payload-shape gaps after the
getting-started handoff. `project_creation.dsn_copied` carried platform
and variant but not `project_id`, while the getting-started removal
events populated their `platform` field with the project slug.

A small `docsFlowProjectIdParams` helper now adds `project_id` to
DSN-copy events for the project-creation and unmarked project surfaces.
The onboarding and SCM-onboarding arms keep their existing payload
shape, matching the established rule that their distinct event names
remain unchanged.

`PlatformDocHeader` now reports `platform.id` for data-removal
confirmation and completion instead of `recentCreatedProject.slug`. The
callback dependency and regression expectations move with that value so
the event contract consistently identifies the SDK platform rather than
the project.

Focused coverage locks the four docs-flow project-ID cases, the unmarked
fallback, both onboarding exclusions, and the corrected platform payload
for marked and unmarked getting-started views.

Refs VDY-133
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.

2 participants