Skip to content

[17.0][FIX] fastapi: Change circular import fix to avoid crash with fastapi>=0.123.7#577

Merged
OCA-git-bot merged 1 commit into
OCA:17.0from
sygel-technology:17.0-fix-fastapi-TYPE_CHECKING-crash
Dec 9, 2025
Merged

[17.0][FIX] fastapi: Change circular import fix to avoid crash with fastapi>=0.123.7#577
OCA-git-bot merged 1 commit into
OCA:17.0from
sygel-technology:17.0-fix-fastapi-TYPE_CHECKING-crash

Conversation

@Tisho99

@Tisho99 Tisho99 commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

Backport of #574

…>=0.123.7

Remove TYPE_CHECKING import condition and delay dependencies import to solve the circular import
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @lmignon,
some modules you are maintaining are being modified, check this out!

@Tisho99 Tisho99 marked this pull request as ready for review December 9, 2025 12:12
@lmignon

lmignon commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-577-by-lmignon-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit b61e2ba into OCA:17.0 Dec 9, 2025
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 6aa9e9a. Thanks a lot for contributing to OCA. ❤️

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