Re: [PR] feat(echarts): allow configuring minInterval on bar and line chart axes [superset]

["rusackas (via GitHub)" <gi...@apache.org>]

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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org