Skip to content

Adds telemetry for walkthrough completion #3723

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Nov 8, 2024
Merged

Adds telemetry for walkthrough completion #3723

merged 2 commits into from
Nov 8, 2024
+212 −64

Conversation

d13
Copy link
Member

@d13 d13 commented Nov 8, 2024 •
edited
Loading

Two updates to note:

  • Some completion events needed to check for a combination of both subscription state and shown/command usage
  • Telemetry events now fire when that completion criteria is met

@d13 d13 self-assigned this Nov 8, 2024
@d13 d13 marked this pull request as draft November 8, 2024 01:19
@d13 d13 marked this pull request as ready for review November 8, 2024 02:56
@d13 d13 force-pushed the feature/walkthrough-telemetry branch from 9783255 to 3c10148 Compare November 8, 2024 14:58
@d13 d13 force-pushed the feature/walkthrough-telemetry branch from 3c10148 to 0dcbe0e Compare November 8, 2024 17:12
@d13 d13 linked an issue Nov 8, 2024 that may be closed by this pull request
Walkthrough (onboarding checklist) revamp #3656
Closed
4 tasks
@d13 d13 merged commit ab6374f into main Nov 8, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@d13 d13

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Walkthrough (onboarding checklist) revamp

1 participant

@d13