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

[16.0][FIX] shift: Recompute partner's can_shop when dependencies change #536

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

carmenbianca
Copy link
Collaborator

@carmenbianca carmenbianca commented Oct 21, 2024

Description

Because this field is a stored computed field, this was previously broken. It wouldn't change, because cooperative_status_ids always remains identical---a single record.

Backport of #535

Odoo task (if applicable)

T13157

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.

Because this field is a stored computed field, this was previously
broken. It wouldn't change, because `cooperative_status_ids` always
remains identical---a single record.

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

@huguesdk to merge

@huguesdk
Copy link
Member

huguesdk commented Nov 1, 2024

/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-536-by-huguesdk-bump-patch, awaiting test results.

@github-grap-bot github-grap-bot merged commit 99c47bd into 16.0 Nov 1, 2024
2 checks passed
@github-grap-bot github-grap-bot deleted the 16.0-fix-can-shop branch November 1, 2024 14:24
@github-grap-bot
Copy link
Contributor

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

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