rusackas commented on PR #33247: URL: https://github.com/apache/superset/pull/33247#issuecomment-3930418411
@sadpandajoe Thanks for the detailed backward compatibility analysis and test suggestions! The tests you proposed have been implemented in commit d43d40da. The tests verify: 1. **Legacy charts** (with only `point_radius_scale` set) receive deck.gl defaults (`getPointRadius=1`, `pointRadiusUnits='meters'`) - preserving existing rendering 2. **New charts** (with all three fields) correctly use the control panel values Also pushed a fix for the pre-commit failure (ee1a8ffa) - the `react/jsx-no-duplicate-props` rule in oxlint doesn't support configuration options, so removed the `ignoreCase` parameter. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
