[GitHub] [superset] graceguo-supercat edited a comment on issue #14056: [alert/report] Solution for dashboards with multiple tabs

[GitBox <gi...@apache.org>]

graceguo-supercat edited a comment on issue #14056:
URL: https://github.com/apache/superset/issues/14056#issuecomment-961286428


   I agree @zuzana-vej's proposal, just 1 small suggestion:
   For case 1 **From dashboard:**
   instead of enforcing report on the current selected tab, give user an option: report default dashboard (by default), or report current selected tab. The UI could be similar to filter component scope configuration:
   <img width="423" alt="Screen Shot 2021-11-04 at 10 11 44 AM" src="https://user-images.githubusercontent.com/27990562/140391976-82c2e54d-a4c0-451c-b2ec-a13cac6cffd2.png">
   This behavior will be consistent with current dashboard report behavior:  by default, report the first tab (and its first children tabs).
   
   Similarly for case 2 **From "Settings > Alerts&Reports"**
   Also add an option: report default dashboard (by default), or select a tab. If user choose select a dashboard tab, then show the dashboard tabs tree. 
   cc @zuzana-vej


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org