sha174n commented on code in PR #38847:
URL: https://github.com/apache/superset/pull/38847#discussion_r3016274240
##########
superset/reports/schemas.py:
##########
@@ -305,6 +305,20 @@ def validate_report_references( # pylint:
disable=unused-argument
)
+class ReportScheduleSubscribeSchema(ReportSchedulePostSchema):
+ """Schema for creating a chart/dashboard subscription.
+
+ ``recipients`` and ``creation_method`` are excluded — both are set
+ server-side: recipients are locked to the authenticated user's email,
+ and creation_method is derived from the presence of ``chart`` or
+ ``dashboard`` in the payload.
+ """
+
+ class Meta:
Review Comment:
Added a type override in ReportScheduleSubscribeSchema restricting to Report
only, with a test covering the Alert rejection.
--
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]
