Skip to content

Conversation

@amoghrajesh
Copy link
Contributor

We do not need this anymore as UI is fastapi + react based.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

We do not need this anymore as UI is fastapi + react based.
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, maybe this should be moved to the fab provider too, because it still seem relevant for it? @vincbeck

@amoghrajesh
Copy link
Contributor Author

Yeah lets wait on Vincent for that input

@vincbeck
Copy link
Contributor

Yes, please move it to FAB provider. I think inside providers/fab/docs/auth-manager makes sense

@amoghrajesh
Copy link
Contributor Author

Alright, will create another PR for that, easier for RM too if core is different from providers.

@amoghrajesh amoghrajesh merged commit 4c71030 into main Apr 21, 2025
51 checks passed
@amoghrajesh
Copy link
Contributor Author

PR: #49492

@kaxil kaxil added this to the Airflow 3.0.0 milestone Apr 21, 2025
kaxil pushed a commit that referenced this pull request Apr 21, 2025
prabhusneha pushed a commit to astronomer/airflow that referenced this pull request Apr 25, 2025
@kaxil kaxil deleted the flask-doc-removal branch April 29, 2025 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants