rusackas commented on PR #44358: URL: https://github.com/apache/superset/pull/44358#issuecomment-5883567913
@FrancescoCastaldi just checking in on this one. The pre-commit failure's cleared up, thanks for that. The other thing from last week is still open though: `touch_dashboards` only fires for newly-added dashboards, so removing a chart from a dashboard still leaves that dashboard's "Last modified" stale, the same symptom #44305 reports. Could you extend the touch to cover removals too? Happy to take another look once that's in. -- 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]
