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 2021/09/08 12:10:57 UTC

[GitHub] [airflow] robinedwards opened a new pull request #18085: Fixes incorrect parameter passed to views (#18083)

robinedwards opened a new pull request #18085:
URL: https://github.com/apache/airflow/pull/18085


   The task_stats, last_dagruns, blocked etc expect dag_ids not dagIds.
   
   This caused the endpoint to return all dags the user had access to by default
   
   closes: #18083


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] github-actions[bot] commented on pull request #18085: Fixes incorrect parameter passed to views (#18083)

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #18085:
URL: https://github.com/apache/airflow/pull/18085#issuecomment-915602744


   The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil merged pull request #18085: Fixes incorrect parameter passed to views (#18083)

Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #18085:
URL: https://github.com/apache/airflow/pull/18085


   


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] ashb commented on pull request #18085: Fixes incorrect parameter passed to views (#18083)

Posted by GitBox <gi...@apache.org>.
ashb commented on pull request #18085:
URL: https://github.com/apache/airflow/pull/18085#issuecomment-915248242


   @robinedwards LGTM -- I'm guessing this should be included in 2.1.4 too?


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] robinedwards commented on pull request #18085: Fixes incorrect parameter passed to views (#18083)

Posted by GitBox <gi...@apache.org>.
robinedwards commented on pull request #18085:
URL: https://github.com/apache/airflow/pull/18085#issuecomment-915273774


   If there is going to be a 2.1.4 that would be much appreciated :-)


-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org