maps icon indicating copy to clipboard operation
maps copied to clipboard

setTelemetryEnabled seems to be deprecated in v10

Open Andarius opened this issue 3 years ago • 4 comments

Describe the bug
MapboxGL.setTelemetryEnabled is undefined with the new Mapbox version

To Reproduce

Call MapboxGL.setTelemetryEnabled(false) from the latest version

Looking at https://www.mapbox.com/telemetry/#opt-out and https://github.com/mapbox/mapbox-maps-android/issues/772 it seems that we cannot disable the telemetry on behalf of the user anymore.

Should setTelemetryEnabled be removed from the MapboxGL namespace ?

Andarius avatar Jun 29 '22 14:06 Andarius

For this version it should return a warning that .setTelemetryEnabled is not supported on v10, and will be removed in the future, PR's Wellcome.

mfazekas avatar Jul 08 '22 11:07 mfazekas

@mfazekas What is the reason for removing this call? We need to comply with GDPR so this was an important function for us. The native SDK's still appear to support this.

beisert1 avatar Aug 18 '22 12:08 beisert1

Adding to what's been said, an option to disable telemetry is also part of MapBox's terms:

Developers employing our Maps SDKs for iOS and Android are required by Mapbox to provide the ability for users to opt out of location telemetry reporting and must provide a location opt-out feature within the settings of any native app.

https://www.mapbox.com/telemetry

This means v10 can't be used without violating MapBox's terms of use.

@mfazekas Has this capability been added somewhere else or why is the method being removed?


Edit:

Actually, what I said isn't quite true. Looking a little more at MapBox's docs:

The default attribution control includes an opt out button. If you hide the attribution control, you must provide an alternative opt out method your users can use.

So it should still be possible to allow users to disable telemetry, but as @beisert1 mentioned, that wont be enough for things like GDPR.

traviswimer avatar Aug 27 '22 01:08 traviswimer

Same here, working on a project with a no-default-data-sharing requirement. Wrote Mapbox a mail to clarify the Terms of Use in context of setting MGLMapboxMetricsEnabled to false while still offering the user the possibility to opt-in again.

rastapasta avatar Aug 30 '22 15:08 rastapasta

I see that this issue is closed and that a PR was merged, but I am hoping to get a bit of clarification. I am migrating from V8 and it appears to me that Mapbox.setTelemetryEnabled(false) still throws 'undefined is not a function' on Android. I believe that Mapbox is the V10 namespace....what am I missing?

NTKrieger avatar Apr 12 '23 19:04 NTKrieger

@NTKrieger see #2789

mfazekas avatar Apr 12 '23 21:04 mfazekas