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.

Sign up for GitHub

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

Jump to bottom

Better telemetry setting UI + clearer wording surrounding new setting #135030

Merged
merged 5 commits into from
Oct 14, 2021
Merged

Better telemetry setting UI + clearer wording surrounding new setting #135030

merged 5 commits into from
Oct 14, 2021

Commits on Oct 13, 2021

  1. Address #134660

    @lramos15
    lramos15 committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    84de861 View commit details
    Browse the repository at this point in the history

  2. Further address #134660

    @lramos15
    lramos15 committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    1f91651 View commit details
    Browse the repository at this point in the history

  3. Clearer Settings UI for new telemetry setting (#135028) 

    * Work on improving telemetry setting description

* two options

* third option

* Add old crash reporter to getTelemetryLevel

* More verbose wording + comments

Co-authored-by: SteVen Batten <sbatten@microsoft.com>
    @lramos15 @sbatten
    lramos15 and sbatten committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    e37c10f View commit details
    Browse the repository at this point in the history

  4. Update telemetry text

    @lramos15
    lramos15 committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    d5654c6 View commit details
    Browse the repository at this point in the history

Commits on Oct 14, 2021

  1. fix strings

    @sbatten
    sbatten committed Oct 14, 2021
    Configuration menu
    Copy the full SHA
    3c133b7 View commit details
    Browse the repository at this point in the history