shahar1 commented on PR #40703: URL: https://github.com/apache/airflow/pull/40703#issuecomment-2266134478
> @shahar1, fyi I've included this change to skip the Dag Run permissions (from dag level) in admin role [f182af9](https://github.com/apache/airflow/commit/f182af9484c1e88cfb662c027da57a759bbea73e) . I think it will fix the db test issue. Apologies for the late response: 1. Could you please merge again from `main` so we could ensure all checks pass? 1. Only now I realized that there's no documentation nor newsfragment includede in this PR. I think that both are essential for other people being aware of this feature, and using it properly. Could you please update the docs. `docs/apache-airflow-providers-fab/auth-manager/access-control.rst` and create `newsframent/40703.feature.rst`? -- 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]
