rusackas commented on PR #43515: URL: https://github.com/apache/superset/pull/43515#issuecomment-5647754665
Thanks for this, the minInterval control is a nice addition. Looks like it needs a rebase (conflicts with master right now), and one of the jest shards is failing in CI. The bot comments about the horizontal bar swap look right to me too: minInterval doesn't get swapped between axes like the other values do for horizontal bars, and the new yVisibility test seems to assert the opposite of what the code actually does. Can you take a look at those before we merge? -- 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]
