Skip to content

Add walkthrough telemetry command #4138

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.

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

Merged
merged 1 commit into from
Aug 17, 2022
Merged

Add walkthrough telemetry command #4138

merged 1 commit into from
Aug 17, 2022

Conversation

andyleejordan
Copy link
Member

This exposes a command that could be used in a walkthrough to send a level (presumably 1, 2, or 3) indicating satisfaction. It also refactors slightly to de-duplicate checking the extension mode before sending telemetry.

This exposes a command that could be used in a walkthrough to send a
level (presumably 1, 2, or 3) indicating satisfaction. It also refactors
slightly to de-duplicate checking the extension mode before sending
telemetry.
@andyleejordan andyleejordan added the Ignore Exclude from the changelog. label Aug 17, 2022
@andyleejordan andyleejordan marked this pull request as ready for review August 17, 2022 17:44
@andyleejordan andyleejordan requested a review from a team August 17, 2022 17:44
Copy link
Collaborator

@SeeminglyScience SeeminglyScience left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@andyleejordan andyleejordan enabled auto-merge (squash) August 17, 2022 17:57
@andyleejordan andyleejordan merged commit 1019661 into main Aug 17, 2022
@andyleejordan andyleejordan deleted the andschwa/telemetry branch August 17, 2022 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ignore Exclude from the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants