Skip to content

[18.0][FIX] fastapi: LGPL-3 header on the two files that carried AGPL-3.0 - #660

Open
lmignon wants to merge 1 commit into
OCA:18.0from
acsone:18.0-fix-licence
Open

lmignon wants to merge 1 commit into
OCA:18.0from
acsone:18.0-fix-licence

Conversation

@lmignon

@lmignon lmignon commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

The module declares LGPL-3 in its manifest and every other file carries the LGPL-3 header. models/ir_rule.py and models/res_lang.py carried the AGPL-3.0 header since they were added, which reads as a copied template rather than a licence choice; this aligns them with the manifest.

backport of #657

The module declares LGPL-3 in its manifest and every other file carries
the LGPL-3 header. models/ir_rule.py and models/res_lang.py carried the
AGPL-3.0 header since they were added, which reads as a copied template
rather than a licence choice; this aligns them with the manifest.
@lmignon lmignon changed the title [FIX] fastapi: LGPL-3 header on the two files that carried AGPL-3.0 [18.0][FIX] fastapi: LGPL-3 header on the two files that carried AGPL-3.0 Oct 16, 2026
@lmignon

lmignon commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot

Copy link
Copy Markdown
Contributor

@lmignon your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-660-by-lmignon-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@lmignon

lmignon commented Oct 16, 2026

Copy link
Copy Markdown
Contributor Author

The tests are failing with Pydantic/FastAPI because we need newer versions of extendable and extendable-pydantic to support the latest versions of Pydantic/FastAPI. See #655, where I included development versions of these dependencies that fix the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants