Vitor-Avila commented on PR #41740: URL: https://github.com/apache/superset/pull/41740#issuecomment-4893789338
@rusackas quick question that might be unrelated with this change: Do you know if this affects reports in any way? Let's say you have a report scheduled to ignore cache, and auto-refresh set to 10s, and your chart ends up taking 15s to load. I'm wondering if the report would send a loading icon (if the auto-refresh was previously not applied to this view). This might also be a concern when directly accessing the dashboard: Does the auto-refresh schedule counts from the moment the dashboard initially loads (so charts are still loading) or does it starts counting after the last chart finished loading? Totally understand if this is out of scope for this PR, just wondering if this could introduce issues to reports. -- 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]
