mistercrunch commented on PR #32867: URL: https://github.com/apache/superset/pull/32867#issuecomment-2887754499
The 3 points from my earlier comments - if we all agree -> ``` So moving forward and in-scope for this PR: * open street map is the default and works out of the box when firing up Superset with default config. Ideally, when creating an ephemeral environment from this PR, or simply firing docker compose up, deckgl would now work out-of-the-box on top of OSM * Mapbox also enabled, but requires setting up an API key to work * configuration/documentation allowing for Mapbox only, mimicking the behavior prior-to-this-PR. Administrators can disable/hide OSM if they desire ``` point 3 requires a new comment in `./UPDATING.md`, something along the lines of "OpenStreetView has been promoted as the new default for Deck.gl visualization since it can be enabled by default without requiring an API key. If you have Mapbox set up and want to disable OpenStreeView in your environment, please follow the steps documented [here](https://superset.apache.org/docs/configuration/configuring-superset/)" -- 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]
