yousoph commented on PR #42491: URL: https://github.com/apache/superset/pull/42491#issuecomment-5098351646
Thanks for the review. I've scoped this PR down to the core regression fix: the one-line `chartStackTrace` forward in the dashboard `Chart` wrapper, which restores stack-trace / "See more" parity with Explore for failed dashboard charts. I reverted the shared `ErrorMessageWithStackTrace` change that was flagged — the identity-based `message` check had the edge case noted (same value passed as both `subtitle` and `descriptionDetails`), and moving the subtitle behind "See more" app-wide was broader than this regression needs. Keeping the fix minimal and targeted. -- 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]
