Send telemetry for notebook trust prompt selections #12964
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
joyceerhl
requested review from
IanMatthewHuff,
DonJayamanne,
claudiaregio,
rchiodo and
DavidKutu
joyceerhl
commented
Jul 14, 2020
src/client/datascience/constants.ts
Outdated
| RunByLineVariableHover = 'DATASCIENCE.RUN_BY_LINE_VARIABLE_HOVER', | ||
| TrustAllNotebooks = 'DATASCIENCE.TRUST_ALL_NOTEBOOKS', | ||
| TrustNotebook = 'DATASCIENCE.TRUST_NOTEBOOK', | ||
| DoNotTrustNotebook = 'DATASCIENCE.DO_NOT_TRUST_NOTEBOOK' |
There was a problem hiding this comment.
When do we use DATASCIENCE vs DS_INTERNAL?
There was a problem hiding this comment.
THe original intent was DATASCIENCE was used for queries used by PMs I think. DS_INTERNAL was stuff we weren't sure we'd need? I can never remember. @IanMatthewHuff may remember. He implemented this distinction.
There was a problem hiding this comment.
Yeah, it's supposed to be anything that is user triggered / part of a user action was DATASCIENCE. DS_INTERNAL is for things like tracking internal timings. DATASCIENCE is correct here as these are all user actions. The intent was to clarify our MAU numbers to make sure that we were only looking at users who were doing something with the product. For instance, auto starting the server (which can happen just from opening a workspace) was getting users counted in MAU even when they never did anything with our product.
rchiodo
approved these changes
Jul 14, 2020
IanMatthewHuff
requested changes
Jul 14, 2020
IanMatthewHuff
approved these changes
Jul 14, 2020
|
Kudos, SonarCloud Quality Gate passed!
|
Codecov Report
@@ Coverage Diff @@
## master #12964 +/- ##
=========================================
Coverage ? 60.06%
=========================================
Files ? 670
Lines ? 36520
Branches ? 5127
=========================================
Hits ? 21937
Misses ? 13487
Partials ? 1096
Continue to review full report at Codecov.
|
joyceerhl
added a commit
that referenced
this pull request
Jul 14, 2020
* Send telemetry for notebook trust prompt * Send telemetry when prompt is shown
joyceerhl
added a commit
that referenced
this pull request
Jul 15, 2020
karthiknadig
added a commit
that referenced
this pull request
Aug 5, 2020
* Update model.isTrusted on trust change (#12820) (#12823) * Reduce visual complexity of trust prompt (#12839) (#12847) * Port python 2.7 fix to release (#12877) * port color fix on collapse all (#12895) (#12897) * fix a color on collapse all (#12895) * update changelog * Merge fixes into July release (#12889) Co-authored-by: Timothy Ruscica <[email protected]> * Merge more fixes into july release (#12918) Co-authored-by: Joyce Er <[email protected]> * Port trust fixes (#12929) * Fix regressions in trusted notebooks (#12902) * Handle trustAllNotebooks selection * Fix bug where after trusting, UI didn't update * Recover from ENOENT due to missing parent directory when trusting notebook (#12913) * Disable keydown on native cells in untrusted notebooks (#12914) * Hide editor icons when editor is not a notebook (#12934) (#12935) * Check for hideFromUser before activating current terminal (#12942) (#12956) * Check for hideFromUser before activating current terminal * Add tests * Tweak logic * Port final trust fixes for release (#12965) * Only allow Enter / NumpadEnter w/o ctrl/shift/alt (#12939) * Send telemetry for notebook trust prompt selections (#12964) * Fixes for persisting trust (#12950) * Display survey for native notebooks on/after 1st August (#12961) (#12975) Co-authored-by: Joyce Er <[email protected]> Co-authored-by: Joyce Er <[email protected]> * Contains cherry picks, version updates, change log updates (#12983) * Update version and change log * Improve detection when LS is fully loaded for IntelliCode (#12853) * Fix path * Actually fix settings * Add news * Add test * Format * Suppress 'jediEnabled' removal * Drop survey first launch threshold * Wait for client ready * Handle async dispose * Fix the date Co-authored-by: Mikhail Arkhipov <[email protected]> * hide the gather button while a cell is executing (#12984) * Update date (#13002) * remove release notes from the start page (#13032) * Cherry pick, version change and change log update (#13079) * Ensure languageServer value is valid, send event during activate (#13064) * Update change log and version * Activate banner prompt for Pylance (#12817) * Fix path * Actually fix settings * Add news * Add test * Format * Suppress 'jediEnabled' removal * Drop survey first launch threshold * Remove LS experiments * Frequency + tests * Fix test * Update message to match spec * Open workspace for extension rather than changing setting * Fix localization string * Show banners asynchronously * Add experiments * Formatting * Typo * Put back verifyAll * Remove obsolete experiments, add Pylance * Suppress experiment if Pylance is installed * PR feedback Co-authored-by: Jake Bailey <[email protected]> * Update change log as per comments Co-authored-by: Jake Bailey <[email protected]> Co-authored-by: Mikhail Arkhipov <[email protected]> * Port fix the gather survey (#13086) (#13105) * Fix the gather survey (#13086) * fix the gather survey added 'gather stats' telemetry mention the gather comments to update the python ext * oops * fix tests and address comments * update gather stats when resetting the kernel * set globalstate vars to 0 when we open vs code * fix gather stats telemetry * fix tests * fix tests for real Co-authored-by: Joyce Er <[email protected]> Co-authored-by: Ian Huff <[email protected]> Co-authored-by: David Kutugata <[email protected]> Co-authored-by: Don Jayamanne <[email protected]> Co-authored-by: Timothy Ruscica <[email protected]> Co-authored-by: Mikhail Arkhipov <[email protected]> Co-authored-by: Jake Bailey <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As requested by @ronglums 😊
package-lock.jsonhas been regenerated by runningnpm install(if dependencies have changed).