mistercrunch commented on a change in pull request #6545: Fix deck.gl Polygon
not show
URL:
https://github.com/apache/incubator-superset/pull/6545#discussion_r242372480
##########
File path: superset/assets/src/visualizations/deckgl/utils.js
##########
@@ -89,8 +89,10 @@ export function getBuckets(fd, features, accessor) {
breakPoints.slice(1).forEach((value, i) => {
const range = breakPoints[i] + ' - ' + breakPoints[i + 1];
const mid = 0.5 * (parseInt(breakPoints[i], 10) + parseInt(breakPoints[i +
1], 10));
+ //fix polygon cannot show
Review comment:
Our linter is picky and wants a space after `//`, otherwise LGTM. You can
run the linter locally with `npm run lint`
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
