o-nikolas commented on code in PR #33971: URL: https://github.com/apache/airflow/pull/33971#discussion_r1312325638
########## docs/apache-airflow-providers-apprise/connections.rst: ########## @@ -20,7 +20,8 @@ Apprise Connection ======================= -The Apprise connection type enables connection to multiple services to send notifications. The complete list of services supported can be found on the `Apprise Wiki <https://github.com/caronc/apprise/wiki#notification-services>`_. +The `Apprise <https://github.com/caronc/apprise>`__ connection type enables connection to multiple services to send notifications. +The complete list of services supported can be found on the `Apprise Wiki <https://github.com/caronc/apprise/wiki#notification-services>`_. Review Comment: ```suggestion The `Apprise <https://github.com/caronc/apprise>`__ connection type enables connection to multiple services to send notifications. The complete list of services supported can be found on the `Apprise Wiki <https://github.com/caronc/apprise/wiki#notification-services>`_. ``` ########## docs/apache-airflow-providers-amazon/connections/chime.rst: ########## @@ -20,7 +20,9 @@ Amazon Chime Connection ========================== -The Chime connection works with calling Chime webhooks to send messages to a chime room. +`Amazon Chime <https://aws.amazon.com/chime/>`__ connection works with calling +`Chime Incoming Webhooks <https://docs.aws.amazon.com/chime/latest/ag/webhooks.html>`__ to send messages to a Review Comment: This was worded incorrectly, even in the old version, not your changes specifically. But since you're here making changes anyway: ```suggestion `Amazon Chime <https://aws.amazon.com/chime/>`__ connection works with `Chime Incoming Webhooks <https://docs.aws.amazon.com/chime/latest/ag/webhooks.html>`__ to send messages to a ``` ########## docs/apache-airflow-providers-apprise/connections.rst: ########## @@ -29,15 +30,36 @@ Apprise hooks point to ``apprise_default`` by default. Configuring the Connection -------------------------- -config (required) - The service(s) to send notifications can be specified here. The format to specify single or multiple services is as follows:: - .. code-block:: python +config (required): + The service(s) to send notifications can be specified here. + The format to specify single or multiple services is as follows Review Comment: Nit: ```suggestion The format to specify a single service or multiple services is as follows: ``` -- 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]
