rusackas commented on PR #38035: URL: https://github.com/apache/superset/pull/38035#issuecomment-3986440527
OK, trying again with the showtime labels 🫠It does seem there are also a lot of code comments you might want to address or refute. I think my main point of nervousness here is still that this doubles the number of plugins, whereas it'd be ideal to either have the plugins (the newer one?) have an option to still support Mapbox as a choice and/or provide a db migration to migrate MapBox utilization over to MapLibre so we can safely remove the legacy plugins. -- 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]
