[IMP] pydantic: moving from pydantic v1 to v2

[petrus-v]

Smothly migrate webservice from pydantic<2.0 using pydantic odoo module to pydantic>2.0 getting the same behaviour retrieves odoo data from Odoo records.

Using this PR to migrate your pydantic models you have to do something this:

• before using pydantic < "2.0" your model looks like something like this:

from pydantic import BaseModel

from odoo.addons.pydantic.utils import GenericOdooGetter


class PartnerModel(BaseModel):
    class Config:
        orm_mode = True
        getter_dict = GenericOdooGetter
    
    id: int
    name: str

• after using pydantic > "2.0" you should subclass PydanticOdooBaseModel which set the pydantic model config with the new from_attributes=True (aka old orm_mode). Your model will get same behaviour as the previous versions using the GenericOdooGetter:

from odoo.addons.pydantic.utils import PydanticOdooBaseModel


class PartnerModel(PydanticOdooBaseModel):

    id: int
    name: str

[OCA-git-bot]

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

[lmignon]

@petrus-v Thank you for the PR. IMO it's not a good idea to try to keep the compatibility with pydantic 1. I'm not in favour of maintaining this support into OCA addon. Pydantic V1 is no more supported with fastapi, extendable_fastapi and a restriction is declared into the manifest file for these addons.

[petrus-v]

@lmignon thanks for your feedback. I'm fine to drop the support to pydantic v1, at least I did it to write some unit test in the module to make sure serialisation behave in the same way. As far I understand your are not using the pydantic module in your v17.0 projects so we can drop the compatibility !

Let's do it !

[petrus-v]

@lmignon et voilà : dropping support to pydantic v1 !

[petrus-v]

@alexandregaldeano, @Koudja I would love to get your review on this one !

[lmignon]

Thank you @petrus-v

[alexandregaldeano]

LGTM

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[lmignon]

/ocabot merge minor

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-551-by-lmignon-bump-minor, awaiting test results.

[OCA-git-bot]

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

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.

[petrus-v]

@lmignon I've pushed a new commit that freeze graphql_server dependency to make the CI check pass to help resuming the merge process

[damdam-s]

works perfectly on custom project !
thanks !

[lmignon]

/ocabot merge minor

[OCA-git-bot]

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

[OCA-git-bot]

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