Skip to content

[18.0][MIG] sale_semaphore: Migration to 18.0#4246

Merged
OCA-git-bot merged 3 commits intoOCA:18.0from
Tecnativa:18.0-MIG-sale_semaphore
Apr 6, 2026
Merged

[18.0][MIG] sale_semaphore: Migration to 18.0#4246
OCA-git-bot merged 3 commits intoOCA:18.0from
Tecnativa:18.0-MIG-sale_semaphore

Conversation

@CarlosRoca13
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@christian-ramos-tecnativa christian-ramos-tecnativa left a comment

Choose a reason for hiding this comment

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

Functional and code review OK!, can you add a ROADMAP with the next lines:

  • Propagate category semaphore recursively to child categories
  • Use pricelist price as the base for calculations

Thanks!

@CarlosRoca13 CarlosRoca13 force-pushed the 18.0-MIG-sale_semaphore branch from 176150b to 5741116 Compare April 6, 2026 11:34
@CarlosRoca13
Copy link
Copy Markdown
Contributor Author

Done @christian-ramos-tecnativa

@pedrobaeza pedrobaeza added this to the 18.0 milestone Apr 6, 2026
@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-4246-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 418373f into OCA:18.0 Apr 6, 2026
7 of 9 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-MIG-sale_semaphore branch April 6, 2026 15:03
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.

5 participants