feluelle commented on a change in pull request #7451: [AIRFLOW-6830] Add
Subject/MessageAttributes to SNS hook and operator
URL: https://github.com/apache/airflow/pull/7451#discussion_r380844082
##########
File path: airflow/providers/amazon/aws/operators/sns.py
##########
@@ -33,8 +33,13 @@ class SnsPublishOperator(BaseOperator):
:type target_arn: str
:param message: the default message you want to send (templated)
:type message: str
+ :param subject: the message subject you want to send (templated)
+ :type subject: str
+ :param message_attributes: the message attributes you want to send as a
flat dict (data type will be
+ determined automatically)
+ :type message_attributes: mapping
Review comment:
`mapping` is not a valid type. You can explain the mapping in detail in the
description. The type should be valid.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
