veeceey commented on PR #37987: URL: https://github.com/apache/superset/pull/37987#issuecomment-3931941307
Thanks for the approval @alexandrusoare! I don't currently have a local Superset dev environment set up to grab a screenshot since the changes are in the query builder layer (not visual). The fix ensures that when `x_axis` is set, the `granularity_sqla` temporal column is excluded from the `GROUP BY` clause, so the visible difference would be in the generated SQL query rather than the chart UI itself. If you'd still like a visual, I can spin up the dev environment and show the SQL output before/after — let me know! -- 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]
