Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Telemetry#obtainTelemetry should be public #12012

Closed
tobrun opened this issue May 25, 2018 · 1 comment · Fixed by #12024
Closed

Telemetry#obtainTelemetry should be public #12012

tobrun opened this issue May 25, 2018 · 1 comment · Fixed by #12024
Labels
Android Mapbox Maps SDK for Android

Comments

@tobrun
Copy link
Member

tobrun commented May 25, 2018

Telemetry#obtainTelemetry is package private while it should be public. End developers need to be able to roll their own custom attribution. As part of that, they need to allow end users to opt-in or out of telemetry collection. To do this they need a reference to Telemetry which is not possible atm.

@tobrun tobrun added the Android Mapbox Maps SDK for Android label May 25, 2018
@tobrun tobrun added this to the android-v6.2.0 milestone May 25, 2018
@tobrun
Copy link
Member Author

tobrun commented May 29, 2018

In #12024 I opted to limit the scope solely to address the issue from OP by exposing 2 configurations instead of exposing the whole object. cc @zugaldia

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

Successfully merging a pull request may close this issue.

1 participant