Skip to content

[16.0] fix se mark to update#212

Merged
OCA-git-bot merged 1 commit into
OCA:16.0from
akretion:16.0-fix-se_mark_to_update
Sep 1, 2025
Merged

[16.0] fix se mark to update#212
OCA-git-bot merged 1 commit into
OCA:16.0from
akretion:16.0-fix-se_mark_to_update

Conversation

@sebastienbeau
Copy link
Copy Markdown
Member

@sebastienbeau sebastienbeau commented Jun 6, 2025

Fix issue : OCA/sale-channel#15

Mark update should not change the state of binding that are planned to be deleted

Check the last commit : @qgroulard

Depends on (include commit):

@sebastienbeau sebastienbeau changed the title 16.0 fix se mark to update [16.0] fix se mark to update Jun 6, 2025
Copy link
Copy Markdown
Contributor

@qgroulard qgroulard left a comment

Choose a reason for hiding this comment

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

LGTM (code review).

Probably my proposition was a bit overkill, this simple fix makes the job.

@sebastienbeau sebastienbeau added this to the 16.0 milestone Aug 31, 2025
@sebastienbeau sebastienbeau force-pushed the 16.0-fix-se_mark_to_update branch from 07e571c to 09f4875 Compare September 1, 2025 09:57
@sebastienbeau
Copy link
Copy Markdown
Member Author

I rebase it, should be ready for merge

@sebastienbeau
Copy link
Copy Markdown
Member Author

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 16.0-ocabot-merge-pr-212-by-sebastienbeau-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 62cfba1 into OCA:16.0 Sep 1, 2025
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

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