Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] account_customer_wallet: Prevent error when creating partner #306

Merged
merged 1 commit into from
Sep 7, 2023

Conversation

carmenbianca
Copy link
Member

@carmenbianca carmenbianca commented Aug 28, 2023

Description

In Odoo 16, it is mandatory to provide a value for computed fields. If you don't, the creation of NewId objects results in error 'Compute method failed to assign res.partner(<NewId
...>,).customer_wallet_balance'.

Odoo task (if applicable)

https://gestion.coopiteasy.be/web#id=11080&action=475&active_id=264&model=project.task&view_type=form&menu_id=536

Checklist before approval

  • Tests are present (or not needed).
  • Credits/copyright have been changed correctly.
  • Change log snippet is present.
  • (If a new module) Moving this to OCA has been considered.

In Odoo 16, it is mandatory to provide a value for computed fields. If
you don't, the creation of NewId objects results in error 'Compute
method failed to assign res.partner(<NewId
...>,).customer_wallet_balance'.

Signed-off-by: Carmen Bianca BAKKER <[email protected]>
@carmenbianca
Copy link
Member Author

/ocabot merge patch

@github-grap-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-306-by-carmenbianca-bump-patch, awaiting test results.

@github-grap-bot github-grap-bot merged commit 4637515 into 16.0 Sep 7, 2023
2 checks passed
@github-grap-bot
Copy link
Contributor

Congratulations, your PR was merged at 6ff8010. Thanks a lot for contributing to coopiteasy. ❤️

@github-grap-bot github-grap-bot deleted the 16.0-wallet-fix-default-value branch September 7, 2023 14:23
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