rusackas commented on PR #41859: URL: https://github.com/apache/superset/pull/41859#issuecomment-5420331739
Looking at the diff again — the fallback for missing `xAxisForceCategorical` only landed in `MixedTimeseries/transformProps.ts`, not the plain `Timeseries/transformProps.ts` that the regular Bar chart uses. So despite the "fixed" note on that bot thread, existing/legacy charts without that field would still hit the regression this PR is meant to prevent. Once that's added and the branch is rebased (still conflicting, and lint-frontend/pre-commit are red), I'm happy to take another look. -- 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]
