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.

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

disable stats #12

Merged
merged 1 commit into from
Jun 17, 2016
Merged

Conversation

nathantsoi
Copy link
Contributor

currently, we're using the cleanflight google analytics tracking id, but i dont think we want to send stats to some place we have no permission to view them :)

this has bugged me for a long time, but i don't think it's cool 1 person can see everyones hardware uuids. (also feature usage, computer properties, etc.) if we value privacy, we should merge this. alternatively, making these stats public would be fine as well.

screen shot 2016-06-15 at 10 32 35 pm

i usually just remove the google analytics permission from my local copy, but this should fixup the issue for everyone.

if we do ever want (hopefully open) analytics, just git revert this commit and update the ga tracking id.

@nathantsoi
Copy link
Contributor Author

this is super easy to revert, so i suggest we merge and revert if/when we want to bring it back, to avoid sending any unnecessary data to google w/ the cleanflight tracking id in the time being

@KiteAnton
Copy link
Contributor

Yes, merge it. It is of no use at the moment and when the need for analytics arise we can revert this then.

@skaman82
Copy link
Contributor

ok, merging

@skaman82 skaman82 merged commit e929a2e into betaflight:development Jun 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants