mistercrunch commented on PR #32867: URL: https://github.com/apache/superset/pull/32867#issuecomment-2814373414
Triggering CI and generating an ephemeral environment to test the feature. Wondering if we should make this the default given that deckgl+mapbox doesn't work out of the box. Would be a much better experience for people trying out Superset to have an actual map out of the box without having to get/configure a mapbox api key. Might make sense to set this new default but make it configurable for those who have an API key and don't want openstreetmap as the default. Would require a new default and a note in `UPDATING.md` so let admins know that the default has changed and that they might want to configure the options. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
