Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Add new "automatically send usage stats" setting under advanced settings #4843

Merged
merged 1 commit into from
Oct 17, 2016
Merged

Add new "automatically send usage stats" setting under advanced settings #4843

merged 1 commit into from
Oct 17, 2016

Conversation

bsclifton
Copy link
Member

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests). N/A
  • Ran git rebase -i to squash commits (if needed).

Add new "automatically send usage stats" setting under advanced settings

Currently, nothing uses this new setting. But now that it's there, it can be used by future changes.

Fixes #4691

Auditors: @aekeus @bradleyrichter

Test Plan:

  1. Open Brave and launch preferences
  2. Go under advanced and see new setting

screen shot 2016-10-16 at 10 02 06 am

@bradleyrichter
Copy link
Contributor

Sending usage statistics should be off by default. Soon we will invite users to enable it on first launch. Otherwise, it looks OK for 1.0.

We may want to remove all of the "(requires browser restart)" and switch to an "*" with footnote at the bottom, after the list.

…ngs.

By default, it's set to false; folks can opt-in by enabling it.

Fixes #4691

Auditors: @aekeus @bradleyrichter

Test Plan:
1. Open Brave and launch preferences
2. Go under advanced and see new setting (set to false)
3. Set to true and quit Brave
4. Relaunch Brave and open preferences > advanced
5. Setting should be persisted
@bsclifton
Copy link
Member Author

Updated to be off by default 😄

@aekeus
Copy link
Member

aekeus commented Oct 17, 2016

++ lgtm

sendCrashReports=Send anonymous crash reports to Brave (requires browser restart)
sendUsageStatistics=Automatically send usage statistics to Brave
Copy link
Member

Choose a reason for hiding this comment

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

@bradleyrichter should we add anonymized or something like that?

@bbondy
Copy link
Member

bbondy commented Oct 17, 2016

++ thanks!

@bbondy
Copy link
Member

bbondy commented Oct 17, 2016

merging for 0.12.6

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants