Skip to content

Toggling the Telemetry opt-out option has no effect #3943

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

Closed
sivaraam opened this issue Sep 28, 2020 · 2 comments · Fixed by #3963
Closed

Toggling the Telemetry opt-out option has no effect #3943

sivaraam opened this issue Sep 28, 2020 · 2 comments · Fixed by #3963
Labels

Comments

@sivaraam
Copy link
Member

Summary:
In the app settings, when I'm unable to toggle the "Telemetry opt-out" option that's supposed to enable/disable the Mapbox telemetry. How many ever times I tap on it, it just remains enabled.

Also, in retrospect, I believe "Telemetry Opt Out" is kind of a contradictory name for the option as it contradicts the option description by conveying that enabling the setting means the user has opted out of telemetry while that is not the case. To my understanding, enabling the setting means telemetry would be sent to Mapbox. So, we should use a better name for the setting too. My suggestion would be "Mapbox Telemetry".

Steps to reproduce:

  1. Open Settings
  2. Scroll and tap on the "Telemetry Opt Out" option

System logs:
Couldn't find any relevant logs in logcat.

Device and Android version:
Samsung SM-J111F running Android 5.1.1.

Commons app version:
2.13.1.793~66f6e2e

@misaochan misaochan added the bug label Sep 30, 2020
@misaochan
Copy link
Member

Can confirm, unable to toggle it off too on real device running Android 10.

To my understanding, enabling the setting means telemetry would be sent to Mapbox. So, we should use a better name for the setting too. My suggestion would be "Mapbox Telemetry".

I agree, but AFAIK we are simply following Mapbox's own requirements. If we are allowed to change it, that would be great. Should probably be a separate issue though. ;)

@sivaraam
Copy link
Member Author

sivaraam commented Oct 1, 2020

Should probably be a separate issue though. ;)

You're right. I opened #3946 for this. We can continue the discussion there.

ashishkumar468 added a commit to ashishkumar468/apps-android-commons that referenced this issue Oct 11, 2020
* Fix telemetry opt-out preference toggler
misaochan pushed a commit that referenced this issue Oct 12, 2020
* Fix telemetry opt-out preference toggler
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 a pull request may close this issue.

2 participants