Skip to content

Conversation

tonyanziano
Copy link
Contributor

Fixes #2035

===

  • Added toolName: 'bf-emulator' to all outgoing events so that we can filter them out from other tools (bf-cli) that use the same Azure Application Insights instance
  • removed livechat_openRemote event and added livechat_open which tracks both local and remote URLs being opened
  • added tracking for whether a livechat opened is for gov cloud or not
  • added tracking for whether a livechat opened is in debug mode or not
  • consolidated Azure signin events (signIn_success and signIn_failure) into one event (azure_signIn) that now tracks success and persistLogin
  • added an event (ngrok_reconnect) for when the user reconnects to ngrok
  • added an event (app_changeSetting) that tracks whenever a single or multiple app settings are changed at one time
  • added an event (oauth_sendToken) that tracks when an OAuth flow is completed and a token is sent to the bot

@coveralls
Copy link

coveralls commented Jan 27, 2020

Coverage Status

Coverage increased (+0.08%) to 68.062% when pulling f43eb80 on toanzian/telemetry into 8201114 on master.

@@ -125,7 +125,7 @@ describe('TelemetryService', () => {
});

it('should track events', () => {
Object.assign(global, { __JEST_ENV__: false });
global['__JEST_ENV__'] = false;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I couldn't get spread to work for whatever reason

@tonyanziano tonyanziano merged commit d3d9bf5 into master Jan 31, 2020
@tonyanziano tonyanziano deleted the toanzian/telemetry branch January 31, 2020 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Additional telemetry events
3 participants