Skip to content

Conversation

@jedcunningham
Copy link
Member

@jedcunningham jedcunningham commented Feb 23, 2025

[fastapi] base_url expected that the server would be running with the breeze port, but that's not always true. Breeze should override for its specific environment instead of the other way around.

`[fastapi] base_url` expected that the server would be running with the
breeze port, but that's not always true. Breeze should override for it's
specific environment instead of the other way around.
Copy link
Member

@gopidesupavan gopidesupavan left a comment

Choose a reason for hiding this comment

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

nice :)

@ashb ashb merged commit 54863bd into apache:main Feb 23, 2025
89 of 90 checks passed
@ashb ashb deleted the fix_ui_non_breeze branch February 23, 2025 08:15
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
`[fastapi] base_url` expected that the server would be running with the
breeze port, but that's not always true. Breeze should override for it's
specific environment instead of the other way around.
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.

3 participants