leocape commented on a change in pull request #13104:
URL: https://github.com/apache/superset/pull/13104#discussion_r576609031



##########
File path: docs/src/pages/docs/installation/email_reports.mdx
##########
@@ -6,8 +6,309 @@ index: 10
 version: 1
 ---
 
-## Scheduling and Emailing Reports
+## Alerts and Reports

Review comment:
       I'd definitely agree, was really just for organisation so that in the 
menu there isn't 'Alerts and reporting' and then below it 'Scheduling and 
Emailing reports' which would be confusing imo, but if @srinify can help we 
could put it into its own section 'Alerts and Reporting', and then have the 
legacy reporting stuff remain at the bottom of this file?




----------------------------------------------------------------
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to