[GitHub] [incubator-superset] bkyryliuk commented on pull request #9944: feat: Alerts! allowing users to set SQL-based email alerts with screenshots

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

bkyryliuk commented on pull request #9944:
URL: https://github.com/apache/incubator-superset/pull/9944#issuecomment-651314045


   @mistercrunch, @robdiciuccio how would you prefer to take it forward? 
   This PR touches many files and it would be hard to keep it up to date if this one doesn't get merged.
   From my prospective we would love to see it merged and contribute towards alerting in superset, e.g. we would start with adding tests & making it more robust.


----------------------------------------------------------------
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.

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