vincbeck commented on code in PR #47571: URL: https://github.com/apache/airflow/pull/47571#discussion_r1987425137
########## providers/fab/tests/unit/fab/www/views/test_views_custom_user_views.py: ########## @@ -26,7 +26,7 @@ from airflow import settings from airflow.providers.fab.www import app as application -from airflow.security import permissions +from airflow.providers.fab.www.security import permissions Review Comment: Not especially related. I can create a separate PR just for this if you prefer. Permissions are duplicated in core airflow and FAB provider (ideally it should be removed from core Airflow but that requires a bit of work). In FAB provider, let's use permissions from FAB provider -- 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]
