vincbeck commented on code in PR #48968:
URL: https://github.com/apache/airflow/pull/48968#discussion_r2035406705


##########
providers/fab/src/airflow/providers/fab/auth_manager/security_manager/override.py:
##########
@@ -729,6 +735,15 @@ def auth_role_admin(self):
         """Get the admin role."""
         return self.appbuilder.get_app.config["AUTH_ROLE_ADMIN"]
 
+    @property
+    @deprecated(

Review Comment:
   I know ... I can remove the deprecation warning. I hesitated. The reason why 
is that it was present in AF2, I just cherry-picked some code from AF2 branch 
and this method was deprecated. But I agree, this feels weird



-- 
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]

Reply via email to