mistercrunch commented on code in PR #32300:
URL: https://github.com/apache/superset/pull/32300#discussion_r1960520175
##########
superset-frontend/plugins/legacy-plugin-chart-country-map/src/CountryMap.js:
##########
@@ -37,7 +37,6 @@ const propTypes = {
width: PropTypes.number,
height: PropTypes.number,
country: PropTypes.string,
- colorScheme: PropTypes.string,
Review Comment:
Do we not support categorical use cases here? I forgot the details, but I
think there was a way to use this visualization to show categorical
classification in some cases. Like plotting categorical for something like
"most grown crop" on a per-state basis.
Seems the color scheme related changes here don't align with the title/body
of the PR (which is about iso code support)
--
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]
