Re: [PR] Simplified configuration parsing in Apprise Connection [airflow]

2024-02-08 Thread via GitHub
hussein-awala commented on PR #37202: URL: https://github.com/apache/airflow/pull/37202#issuecomment-1935042072 Congrats on your first commit 🎉 -- 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 th

Re: [PR] Simplified configuration parsing in Apprise Connection [airflow]

2024-02-08 Thread via GitHub
boring-cyborg[bot] commented on PR #37202: URL: https://github.com/apache/airflow/pull/37202#issuecomment-1935041573 Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.

Re: [PR] Simplified configuration parsing in Apprise Connection [airflow]

2024-02-08 Thread via GitHub
hussein-awala merged PR #37202: URL: https://github.com/apache/airflow/pull/37202 -- 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: commits-unsubscr...@ai

Re: [PR] Simplified configuration parsing in Apprise Connection [airflow]

2024-02-08 Thread via GitHub
infohash commented on PR #37202: URL: https://github.com/apache/airflow/pull/37202#issuecomment-1933613022 No, they don't have to which is why backward compatibility is there. Until now, there was only one way: ```python $ export AIRFLOW_CONN_APPRISE_DEFAULT='{"extra": {"conf

Re: [PR] Simplified configuration parsing in Apprise Connection [airflow]

2024-02-07 Thread via GitHub
dirrao commented on PR #37202: URL: https://github.com/apache/airflow/pull/37202#issuecomment-1933351952 > config Does airflow consumers needs to update the environment variables without escaping the double quotes? -- This is an automated message from the Apache Git Service. To re

Re: [PR] Simplified configuration parsing in Apprise Connection [airflow]

2024-02-07 Thread via GitHub
Taragolis commented on code in PR #37202: URL: https://github.com/apache/airflow/pull/37202#discussion_r1481752995 ## docs/apache-airflow-providers-apprise/connections.rst: ## @@ -60,6 +60,17 @@ Config (required) }, ] -.. seealso:: - - `Apprise URL Bas