rusackas commented on PR #43863: URL: https://github.com/apache/superset/pull/43863#issuecomment-5769565224
Candlestick chart looks great, but there are still three open threads worth clearing before this merges. @sadpandajoe's null-sentinel and tooltip-trigger catches both got a "fixed" reply from you, but the threads are still open so I can't tell those landed on the current head. Copilot's context-menu point on `inContextMenu` never being true for this plugin also hasn't been addressed. I'll see if I can help fix these items. -- 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]
