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] stock_release_channel: do not write records in _compute_picking_count #969

Conversation

sebalix
Copy link
Contributor

@sebalix sebalix commented Jan 9, 2025

This is triggering concurrency updates on release channel records while reading them in the dashboard (updating write_date + write_uid fields).

…unt'

This is triggering concurrency updates on release channel records while
reading them in the dashboard (updating `write_date` + `write_uid`
fields).
@sebalix sebalix added the bug Something isn't working label Jan 9, 2025
@sebalix sebalix added this to the 16.0 milestone Jan 9, 2025
@OCA-git-bot
Copy link
Contributor

Hi @mt-software-de, @jbaudoux,
some modules you are maintaining are being modified, check this out!

@sebalix
Copy link
Contributor Author

sebalix commented Jan 9, 2025

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-969-by-sebalix-bump-patch, awaiting test results.

Copy link
Contributor

@jbaudoux jbaudoux left a comment

Choose a reason for hiding this comment

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

Need to backport this fix on v14
@mt-software-de Fix on #699

@rousseldenis You'll want this also

@OCA-git-bot OCA-git-bot merged commit d9178d4 into OCA:16.0 Jan 9, 2025
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@sebalix
Copy link
Contributor Author

sebalix commented Jan 9, 2025

@jbaudoux @mt-software-de backported in 14.0 here: #970

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved bug Something isn't working merged 🎉
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants