Skip to content
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

Test telemetry #6246

Closed
1 task done
jrieken opened this issue May 10, 2016 · 2 comments
Closed
1 task done

Test telemetry #6246

jrieken opened this issue May 10, 2016 · 2 comments

Comments

@jrieken
Copy link
Member

jrieken commented May 10, 2016

Once again telemetry has undergone some refactorings, esp to fix #1000. The appending of events now happens inside the shared process such that DNS issues don't block the main thread anymore. We should validate that

  • we still send the common properties, like isNewSession, timestamp etc etc
  • we still send the common properties using the same data types, like timesincesessionstart is a number, timestamp is a string
  • we don't miss events on shutdown
  • we still honour the opt out setting
@jrieken jrieken added this to the May 2016 milestone May 10, 2016
@jrieken
Copy link
Member Author

jrieken commented May 10, 2016

@seanmcbreen fyi master of telemetry data

@egamma egamma mentioned this issue May 23, 2016
87 tasks
@weinand
Copy link
Contributor

weinand commented May 25, 2016

@seanmcbreen Joh mentioned that you've already verified this. If this is correct please set your checkbox and close this item.

@weinand weinand closed this as completed May 30, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants