villebro commented on PR #19794: URL: https://github.com/apache/superset/pull/19794#issuecomment-1104797069
A few thoughts: - I've probably raised this many times before, but I'll do it once more, just in case 😆 The title "Chart" (instead of "Query") could be seen as slightly ambiguous, as everything should affect the chart. While I know that "Query" isn't very business user friendly, maybe there's some other term out there that could convey more clearly what these controls are. - I noticed that Mixed Timeseries isn't included in this PR. Could we update that one, too? -- 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]
