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

OpenStreetMap insted of Mapbox #169

Closed
florian-sabonchi opened this issue May 16, 2022 · 5 comments
Closed

OpenStreetMap insted of Mapbox #169

florian-sabonchi opened this issue May 16, 2022 · 5 comments
Labels
duplicate This issue or pull request already exists

Comments

@florian-sabonchi
Copy link

Hello what do you think to use OpenStreet map instead of mapbox

@johan12345 johan12345 changed the title OpenStreet map insted of mapbox OpenStreetMap insted of Mapbox May 16, 2022
@johan12345
Copy link
Collaborator

Mapbox maps are based on OpenStreetMap data and their client SDK (at least the older version that we are currently using) is open source. So what exactly are you proposing and what benefit would that bring?

  • Using a different client library?
  • Using a different OpenStreetMap tile provider?
  • Using OpenStreetMap data for charging stations? (this is currently in development, see Support OpenStreetMap as data source #97)
  • Using a different service for geocoding queries (search field)?

I already wrote some comments on #36 and #141 that might help understand why we are currently using Mapbox, which other options are available and why some of them are not currently feasible.

@florian-sabonchi
Copy link
Author

Hello Thank you for the reply. I was actually not aware of that, I want to avoid unnecessary tracking services. Have you heard of warden? Warden is an open source application that detects trackers within APKs see here:
https://gitlab.com/AuroraOSS/AppWarden

Unfortunately, Warden seems to classify Mapbox as a tracker. I hope you can understand now why I opened this issue

@florian-sabonchi
Copy link
Author

Screenshot_20220516-171638

@florian-sabonchi
Copy link
Author

florian-sabonchi commented May 16, 2022

I have read your comment I think the issue was unnecessary. thanks for your great work

@johan12345
Copy link
Collaborator

Hm okay, yeah, so this tool seems to just detect „trackers“ based on package names, not checking if tracking code is actually included and enabled. As stated in #141 all telemetry is disabled, there are just some concerns about the SKU token which is necessary for billing purposes. So I‘ll close this as a duplicate of #141.

@johan12345 johan12345 added the duplicate This issue or pull request already exists label May 16, 2022
johan12345 added a commit that referenced this issue May 8, 2024
Use Jawg Maps for basemap, ArcGIS for satellite maps

fixes #141
refs #169, #197
johan12345 added a commit that referenced this issue May 8, 2024
Use Jawg Maps for basemap, ArcGIS for satellite maps

fixes #141
refs #169, #197

hide traffic checkbox if traffic is not supported by map
johan12345 added a commit that referenced this issue May 19, 2024
Use Jawg Maps for basemap, ArcGIS for satellite maps

fixes #141
refs #169, #197

hide traffic checkbox if traffic is not supported by map
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants