event icon indicating copy to clipboard operation
event copied to clipboard
verified publisher icon OCA

[17.0][FIX] partner_event: handle contacts with mobile field

Open em230418 opened this issue 1 year ago • 1 comments

In Odoo event.registration.mobile field does not exist.

As result before this commit, if you manually create registration with partner that has "mobile" field set, KeyError will be raised.

Notes for future migrations to 19.0+ included.

em230418 avatar Feb 28 '25 09:02 em230418

codecov/patch — 88.88% of diff hit (target 96.89%)

I don't know, what it asks from me

em230418 avatar Feb 28 '25 10:02 em230418

/ocabot merge patch

pedrobaeza avatar May 07 '25 18:05 pedrobaeza

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

OCA-git-bot avatar May 07 '25 18:05 OCA-git-bot

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

OCA-git-bot avatar May 07 '25 18:05 OCA-git-bot

@em230418 can you fw-port it to upper versions?

pedrobaeza avatar May 07 '25 18:05 pedrobaeza

@em230418 can you fw-port it to upper versions?

Yes, I will. I hope next week.

em230418 avatar May 15 '25 10:05 em230418

@em230418 can you fw-port it to upper versions?

Looks like partner_event is not ported to 18.0 yet, so the only thing I did - mentioned this PR in 18.0 migration PR

em230418 avatar May 17 '25 12:05 em230418