Re: [PR] Add Jira Notifier implementation [airflow]

eladkal merged PR #35397: URL: https://github.com/apache/airflow/pull/35397 -- 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...@airflow.

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on PR #35397: URL: https://github.com/apache/airflow/pull/35397#issuecomment-1811777632 A kind reminder on this. Are we good to merge? -- 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

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1388950665 ## docs/apache-airflow-providers-atlassian-jira/notifications/jira-notifier-howto-guide.rst: ## @@ -0,0 +1,65 @@ + + .. Licensed to the Apache Software Foundation (A

Re: [PR] Add Jira Notifier implementation [airflow]

eladkal commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1388924796 ## docs/apache-airflow-providers-atlassian-jira/notifications/jira-notifier-howto-guide.rst: ## @@ -0,0 +1,65 @@ + + .. Licensed to the Apache Software Foundation (AS

Re: [PR] Add Jira Notifier implementation [airflow]

eladkal commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1388924796 ## docs/apache-airflow-providers-atlassian-jira/notifications/jira-notifier-howto-guide.rst: ## @@ -0,0 +1,65 @@ + + .. Licensed to the Apache Software Foundation (AS

Re: [PR] Add Jira Notifier implementation [airflow]

Taragolis commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1385706929 ## docs/apache-airflow-providers-atlassian-jira/notifications/jira-notifier-howto-guide.rst: ## @@ -0,0 +1,65 @@ + + .. Licensed to the Apache Software Foundation (

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1384405364 ## docs/apache-airflow-providers-atlassian-jira/notifications/jira-notifier-howto-guide.rst: ## @@ -0,0 +1,65 @@ + + .. Licensed to the Apache Software Foundation (A

Re: [PR] Add Jira Notifier implementation [airflow]

eladkal commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1384389827 ## docs/apache-airflow-providers-atlassian-jira/notifications/jira-notifier-howto-guide.rst: ## @@ -0,0 +1,65 @@ + + .. Licensed to the Apache Software Foundation (AS

Re: [PR] Add Jira Notifier implementation [airflow]

Taragolis commented on PR #35397: URL: https://github.com/apache/airflow/pull/35397#issuecomment-1793866976 > the left navigation the version says 2.2.0? That is provider version. Providers release cycle are different then Airflow itself -- This is an automated message from the Ap

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on PR #35397: URL: https://github.com/apache/airflow/pull/35397#issuecomment-1793790042 Locally built docs:  ![image](https://github.com/apache/airflow/assets/6260009/edfe

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on PR #35397: URL: https://github.com/apache/airflow/pull/35397#issuecomment-1793782902 > Some NIT which might make it better > > In general looks good, also check documentation locally Built and checked the new docs locally. There weren't any issues. Will che

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1382605196 ## airflow/providers/atlassian/jira/notifications/jira.py: ## @@ -0,0 +1,83 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor l

Re: [PR] Add Jira Notifier implementation [airflow]

Taragolis commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1382547115 ## airflow/providers/atlassian/jira/notifications/jira.py: ## @@ -0,0 +1,83 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Add Jira Notifier implementation [airflow]

Taragolis commented on PR #35397: URL: https://github.com/apache/airflow/pull/35397#issuecomment-1793580399 Documentation should be pretty simple: copy from one of existed provider, replace by new example (you have in description of PR), change reference to the Notifier classes, build it lo

Re: [PR] Add Jira Notifier implementation [airflow]

pubudu91 commented on PR #35397: URL: https://github.com/apache/airflow/pull/35397#issuecomment-1793318243 @Taragolis Addressed the above mentioned suggestions. Still working on the tests and the docs. -- This is an automated message from the Apache Git Service. To respond to the message

Re: [PR] Add Jira Notifier implementation [airflow]

Taragolis commented on code in PR #35397: URL: https://github.com/apache/airflow/pull/35397#discussion_r1381531187 ## airflow/providers/atlassian/jira/notifications/jira.py: ## @@ -0,0 +1,80 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[PR] Add Jira Notifier implementation [airflow]

pubudu91 opened a new pull request, #35397: URL: https://github.com/apache/airflow/pull/35397 Fixes #35261 Sample usage: ```python from datetime import datetime from airflow import DAG from airflow.operators.bash import BashOperator from airflow.provider