@mahendra Added a few comments, will add some more tomorrow. Some general comments: - With the risk of introducing feature creep, I think it would be good to have a one-off emailing feature in the scheduling UI to test what the report looks like. It would make it easier to test if email settings work correctly, and also check if the email report renders properly. - Another feature that would be valuable in the UI (not for this release, but later): Custom resolution (if a report looks better in some resolution other than that defined in `superset_config.py`), - Outside of the scope of this PR, but `EMAIL_NOTIFICATIONS` is defined twice in `config.py`.
[ Full content available at: https://github.com/apache/incubator-superset/pull/5294 ] This message was relayed via gitbox.apache.org for [email protected]
