You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/06/19 07:56:10 UTC

[GitHub] [airflow] feluelle commented on a change in pull request #9355: Move out sendgrid email from airflow.contrib

feluelle commented on a change in pull request #9355:
URL: https://github.com/apache/airflow/pull/9355#discussion_r442690842



##########
File path: UPDATING.md
##########
@@ -62,6 +62,28 @@ https://developers.google.com/style/inclusive-documentation
 
 -->
 
+### SendGrid emailer has been moved
+
+Formerly the core code was maintained by the original creators - Airbnb. The code that was in the contrib
+package was supported by the community. The project was passed to the Apache community and currently the
+entire code is maintained by the community, so now the division has no justification, and it is only due
+to historical reasons.
+
+To clean up, the `send_mail` function from the `airflow.contrib.utils.sendgrid` module has been moved.
+
+If your configuration file looks like this.
+```ini
+[email]
+email_backend = airflow.contrib.utils.sendgrid.send_email
+```
+It should look like this now:
+```ini
+[email]
+email_backend = airflow.providers.sendgrid.utils.emai.send_email

Review comment:
       ```suggestion
   email_backend = airflow.providers.sendgrid.utils.emailer.send_email
   ```




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