Skip to content

[18.0][MIG] sale_order_split_strategy#3630

Closed
TDu wants to merge 10 commits intoOCA:18.0from
camptocamp:18.0-mig-sale_order_split_strategy
Closed

[18.0][MIG] sale_order_split_strategy#3630
TDu wants to merge 10 commits intoOCA:18.0from
camptocamp:18.0-mig-sale_order_split_strategy

Conversation

@TDu
Copy link
Copy Markdown
Member

@TDu TDu commented Mar 17, 2025

No description provided.

@TDu TDu force-pushed the 18.0-mig-sale_order_split_strategy branch from 2b2219d to 36ba71b Compare March 17, 2025 16:26
@sebalix
Copy link
Copy Markdown
Contributor

sebalix commented Mar 26, 2025

/ocabot migration sale_order_split_strategy

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Mar 26, 2025
@@ -0,0 +1,2 @@
This module provides the possibility to define split strategies to be
applied on sales order for splitting.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This could be enhanced in order to explain in which context using the module.

Comment thread sale_order_split_strategy/readme/CONFIGURE.md Outdated
Comment thread sale_order_split_strategy/models/sale_order.py Outdated
@TDu
Copy link
Copy Markdown
Member Author

TDu commented Apr 2, 2025

@rousseldenis Thanks for the review, indeed poor readme, I tried to improve them a little.

Comment thread sale_order_split_strategy/readme/USAGE.md Outdated
Comment thread sale_order_split_strategy/readme/USAGE.md Outdated
Comment thread sale_order_split_strategy/readme/DESCRIPTION.md Outdated
@TDu TDu force-pushed the 18.0-mig-sale_order_split_strategy branch from ff88d20 to ec79966 Compare April 2, 2025 09:14
@TDu
Copy link
Copy Markdown
Member Author

TDu commented Apr 2, 2025

@twalter-c2c Thanks for the review, I applied most of your comments

@twalter-c2c
Copy link
Copy Markdown
Contributor

Thank you, good job.

@simahawk
Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-3630-by-simahawk-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 14, 2025
Signed-off-by simahawk
@OCA-git-bot
Copy link
Copy Markdown
Contributor

It looks like something changed on 18.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 18.0-ocabot-merge-pr-3630-by-simahawk-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 14, 2025
Signed-off-by simahawk
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at efe711b. 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.

9 participants