Skip to content

Report telemetry events #41

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 6 commits into from
Jun 19, 2020
Merged

Report telemetry events #41

merged 6 commits into from
Jun 19, 2020

Conversation

shinin-light
Copy link
Contributor

Step 3 of: https://deepcodeai.atlassian.net/browse/COL-729
(requires DeepCodeAI/product#505 && snyk/code-client#13)

  • Added telemetry configuration variables
  • Added thin ReportModule
  • Slightly refactored dependency injection through activate function
  • Added ignoreSuggestion event

NOT DONE:

  • viewSuggestion event has been defined but not implemented as we can only act upon hover or codeAction (i.e. Quick fix) events right now. The MVP request is satisfied as both configuration and telemetry pipelines are in place (@deepcodeg). After full implementation of DeepCode tab/panel, we will add custom viewSuggestion triggers and web views.

@shinin-light shinin-light requested a review from Arvi3d June 8, 2020 12:04
@Arvi3d Arvi3d merged commit 024bdab into master Jun 19, 2020
@Arvi3d Arvi3d deleted the report_suggestion_view branch June 19, 2020 11:31
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.

2 participants