vincbeck commented on code in PR #45009: URL: https://github.com/apache/airflow/pull/45009#discussion_r1892548707
########## newsfragments/aip-79.significant.rst: ########## @@ -0,0 +1,11 @@ +Remove Flask App Builder from core Airflow dependencies. + +As part of this change the following breaking changes have occurred: + +- The auth manager interface ``base_auth_manager`` have been updated with some breaking changes: + + - The constructor no longer take ``appbuilder`` as parameter. The constructor takes no parameter + + - A new abstract method ``deserialize_user`` needs to be implemented + + - A new abstract method ``serialize_user`` needs to be implemented Review Comment: Those are not related to this PR but I forgot to mention them when adding them to the interface -- 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]
