BasPH commented on code in PR #28845: URL: https://github.com/apache/airflow/pull/28845#discussion_r1071001711
########## docs/apache-airflow/howto/email-config.rst: ########## @@ -29,7 +29,27 @@ in the ``[email]`` section. subject_template = /path/to/my_subject_template_file html_content_template = /path/to/my_html_content_template_file -You can configure sender's email address by setting ``from_email`` in the ``[email]`` section. +Equivalent environment variables look like: + +.. code-block:: + + AIRFLOW__EMAIL__EMAIL_BACKEND=airflow.utils.email.send_email_smtp + AIRFLOW__EMAIL__SUBJECT_TEMPLATE=/path/to/my_subject_template_file + AIRFLOW__EMAIL__HTML_CONTENT_TEMPLATE=/path/to/my_html_content_template_file + +You can configure a sender's email address by setting ``from_email`` in the ``[email]`` section like: + +.. code-block:: ini + + [email] + from_email = "[email protected]" + +Equivalent environment variables look like: + +.. code-block:: + + AIRFLOW__EMAIL__FROM_EMAIL="From email <[email protected]>" Review Comment: Wow, that's very unexpected, but thanks for clarifying. We could at least make it slightly less confusing by providing an example name, because showing `From email` does not tell the reader the expected value IMO. For example: ``` AIRFLOW__EMAIL__FROM_EMAIL=John Doe <"[email protected]"> ``` -- 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]
