Re: [PR] Add TemplatedSmtpNotifier [airflow]

2023-12-15 Thread via GitHub
vchiapaikeo commented on code in PR #36226: URL: https://github.com/apache/airflow/pull/36226#discussion_r1428161896 ## airflow/providers/smtp/notifications/templated_smtp.py: ## @@ -0,0 +1,51 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributo

Re: [PR] Add TemplatedSmtpNotifier [airflow]

2023-12-15 Thread via GitHub
eladkal commented on code in PR #36226: URL: https://github.com/apache/airflow/pull/36226#discussion_r1427829595 ## airflow/providers/smtp/notifications/templated_smtp.py: ## @@ -0,0 +1,51 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor li

Re: [PR] Add TemplatedSmtpNotifier [airflow]

2023-12-14 Thread via GitHub
vchiapaikeo commented on PR #36226: URL: https://github.com/apache/airflow/pull/36226#issuecomment-1856403567 cc @eladkal , @potiuk , @BasPH, @hussein-awala - would love to hear what you think when you have a moment -- This is an automated message from the Apache Git Service. To respond t

[PR] Add TemplatedSmtpNotifier [airflow]

2023-12-14 Thread via GitHub
vchiapaikeo opened a new pull request, #36226: URL: https://github.com/apache/airflow/pull/36226 closes: https://github.com/apache/airflow/issues/35381 Creating a TemplatedSmtpNotifier class that simplifies sending emails based on the discussion in https://github.com/apach