Skip to content

[18.0][MIG] hr_announcement: Migration to 18.0#1582

Open
ManfredNelvin-Dev wants to merge 13 commits into
OCA:18.0from
ManfredNelvin-Dev:18.0-mig-hr_announcement
Open

[18.0][MIG] hr_announcement: Migration to 18.0#1582
ManfredNelvin-Dev wants to merge 13 commits into
OCA:18.0from
ManfredNelvin-Dev:18.0-mig-hr_announcement

Conversation

@ManfredNelvin-Dev
Copy link
Copy Markdown

Migration to 18.0

@pedrobaeza
Copy link
Copy Markdown
Member

What about #1549? Did you check the problem depicted there?

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot migration hr_announcement

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jun 1, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 1, 2026
23 tasks
@ManfredNelvin-Dev
Copy link
Copy Markdown
Author

Hi @pedrobaeza

Yes, I reviewed #1549. The issue reported there was the use of vals_list.get("employee_ids") in the create() override, which is incorrect because vals_list is a list when using @api.model_create_multi.

In this PR, that has been fixed by checking:
any(vals.get("employee_ids") for vals in vals_list)

The Odoo and OCB test suites are also passing successfully.

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.

7 participants