Skip to content

[FIX] tracking_manager: Fixed wrong usage of create function#3434

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
AmetrasIntelligence:16.0
Nov 7, 2025
Merged

[FIX] tracking_manager: Fixed wrong usage of create function#3434
OCA-git-bot merged 1 commit into
OCA:16.0from
AmetrasIntelligence:16.0

Conversation

@baguenth

@baguenth baguenth commented Nov 3, 2025

Copy link
Copy Markdown
Member

Using the wrong parameter in create function leads to, that globally all create-functions called with parameter "vals_list" will break.

TypeError: Base.create() got an unexpected keyword argument 'vals_list'

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @sebastienbeau, @Kev-Roche,
some modules you are maintaining are being modified, check this out!

@baguenth

baguenth commented Nov 7, 2025

Copy link
Copy Markdown
Member Author

Hey @rvalyi @sebastienbeau @Kev-Roche @pedrobaeza,

can someone take a look? Should be a straight forward fix.

@pedrobaeza pedrobaeza added this to the 16.0 milestone Nov 7, 2025

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's clear, so fast-tracking it:

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit d458099 into OCA:16.0 Nov 7, 2025
9 checks passed
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.

3 participants