Skip to content

[17.0][MIG] sale_order_product_assortment: Migration to 17.0 #3669

Merged
OCA-git-bot merged 24 commits intoOCA:17.0from
solvosci:17.0-mig-sale_order_product_assortment
Jun 27, 2025
Merged

[17.0][MIG] sale_order_product_assortment: Migration to 17.0 #3669
OCA-git-bot merged 24 commits intoOCA:17.0from
solvosci:17.0-mig-sale_order_product_assortment

Conversation

@DantePereyra
Copy link
Copy Markdown
Contributor

@DantePereyra DantePereyra commented Apr 8, 2025 •
edited
Loading

Standard migration

Depends on:

CarlosRoca13 and others added 20 commits April 7, 2025 17:38
@CarlosRoca13 @DantePereyra
[ADD] sale_order_product_assortment: New module to use the product as…
5d821b8 
…sortment

This module allow to use the product assortment related to partners on sale orders.
@CarlosRoca13 @DantePereyra
[IMP] sale_order_product_assortment: black, isort, prettier
7d3a8cb
@CarlosRoca13 @DantePereyra
[MIG] sale_order_product_assortment: Migration to v13
e2bec99
@danimv5 @DantePereyra
Translated using Weblate (Catalan)
0bbdf6e 
Currently translated at 100.0% (9 of 9 strings)

Translation: sale-workflow-13.0/sale-workflow-13.0-sale_order_product_assortment
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-13-0/sale-workflow-13-0-sale_order_product_assortment/ca/
@sergio-teruel @DantePereyra
[IMP] sale_order_product_assortment: Improve domain
9149f55 
Use product_assortment domain and only one field to get allowed products result from whitelist and blacklist

TT30584
@sergio-teruel @DantePereyra
[FIX] sale_order_product_assortment: Make invisible column product_al…
529a592 
…lowed_id

TT33641
@ilyasProgrammer @DantePereyra
[MIG] sale_order_product_assortment: Migration to 14.0
42a7590
@francesco-ooops @DantePereyra
Added translation using Weblate (Italian)
6251741
@CarlosRoca13 @DantePereyra
[IMP] sale_order_product_assortment: black, isort, prettier
2736fe1
@sergio-teruel @DantePereyra
[IMP] product_assortment: Add blacklist product domain to exclude som…
d1b9525 
…e products

TT36814
@CarlosRoca13 @DantePereyra
[MIG] sale_order_product_assortment: Migration to version 15.0
2feca0b
@carlosdauden @DantePereyra
[IMP] sale_order_product_assortment: Use original product_id field ke…
35723a6 
…eping domain
@carlosdauden @DantePereyra
[IMP] sale_order_product_assortment: Increase performance using new g…
3e509d8 
…et_all_partner_ids cached method
@carlosdauden @DantePereyra
[IMP] sale_order_product_assortment: Use new field applied_assortment…
b4c130f 
…s_ids
@Ivorra78 @DantePereyra
Translated using Weblate (Spanish)
2a47dd6 
Currently translated at 100.0% (3 of 3 strings)

Translation: sale-workflow-15.0/sale-workflow-15.0-sale_order_product_assortment
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-15-0/sale-workflow-15-0-sale_order_product_assortment/es/
@carolinafernandez-tecnativa @DantePereyra
[MIG] sale_order_product_assortment: Migration to 16.0
7370953 
TT44355
@DantePereyra
[UPD] Update sale_order_product_assortment.pot
b2505d7
@OCA-git-bot @DantePereyra
[BOT] post-merge updates
2547986
@mymage @DantePereyra
Translated using Weblate (Italian)
fbd59ad 
Currently translated at 100.0% (3 of 3 strings)

Translation: sale-workflow-16.0/sale-workflow-16.0-sale_order_product_assortment
Translate-URL: https://translation.odoo-community.org/projects/sale-workflow-16-0/sale-workflow-16-0-sale_order_product_assortment/it/
@DantePereyra
[IMP] sale_order_product_assortment: pre-commit auto fixes
5e0d282
@DantePereyra DantePereyra force-pushed the 17.0-mig-sale_order_product_assortment branch 2 times, most recently from fec294a to 98d912b Compare April 8, 2025 11:18
@rousseldenis
Copy link
Copy Markdown
Contributor

rousseldenis commented Apr 23, 2025

/ocabot migration sale_order_product_assortment

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Apr 23, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 23, 2025
Open
rousseldenis
rousseldenis reviewed Apr 23, 2025
View reviewed changes
@DantePereyra DantePereyra force-pushed the 17.0-mig-sale_order_product_assortment branch from 21e5a7a to f75bdc0 Compare May 6, 2025 10:26
@DantePereyra DantePereyra force-pushed the 17.0-mig-sale_order_product_assortment branch from f75bdc0 to 96ad75a Compare May 7, 2025 06:14
@DantePereyra
Copy link
Copy Markdown
Contributor Author

DantePereyra commented May 7, 2025

@rousseldenis could you review it again? product_assortment was merged, it's perfect timing!

@sergio-teruel
Copy link
Copy Markdown
Contributor

sergio-teruel commented May 8, 2025

Hi @DantePereyra Can you add this commit db8fb32

@DantePereyra
Copy link
Copy Markdown
Contributor Author

DantePereyra commented May 8, 2025

Hi @sergio-teruel, I saw that, and also your commit d42eb91, which adds security groups and compute_sudo=True on allowed_product_ids. Should I include that commit as well? Bringing only one seems incomplete to me, but I’ll leave that decision up to you. Thanks!

@rousseldenis
Copy link
Copy Markdown
Contributor

rousseldenis commented May 23, 2025

@sergio-teruel Could you answer ? Thanks

mariancuadranetkia and others added 2 commits June 3, 2025 15:37
@mariancuadranetkia @DantePereyra
[FIX] sale_order_product_assortment: Add compute_sudo in has_allowed_…
4b26040 
…products
@sergio-teruel @DantePereyra
[IMP] sale_order_product_assortment: Add security group to only show …
f4c5239 
…assortments to managers

TT51064

Co-Authored-By: Dante Pereyra <dante.pereyra@solvos.es>
@DantePereyra
Copy link
Copy Markdown
Contributor Author

DantePereyra commented Jun 3, 2025

Hi @rousseldenis @sergio-teruel,I’ve finally brought in those commits and made a few adjustments.
Could you please review it again and let me know if everything looks good?
Thanks!

@pedrobaeza pedrobaeza requested a review from rousseldenis June 10, 2025 06:22
@DantePereyra
Copy link
Copy Markdown
Contributor Author

DantePereyra commented Jun 27, 2025

@OCA/sale-workflow-maintainers, could you take a look on this? Thanks!

marcelsavegnago
marcelsavegnago approved these changes Jun 27, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@marcelsavegnago marcelsavegnago left a comment

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented Jun 27, 2025

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza
Copy link
Copy Markdown
Member

pedrobaeza commented Jun 27, 2025

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented Jun 27, 2025

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

@OCA-git-bot OCA-git-bot merged commit a44eadc into OCA:17.0 Jun 27, 2025
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented Jun 27, 2025

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

Reviewers

@rousseldenis rousseldenis Awaiting requested review from rousseldenis

2 more reviewers

@marcelsavegnago marcelsavegnago marcelsavegnago approved these changes

@sergio-teruel sergio-teruel sergio-teruel approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved merged 🎉 migration needs review ready to merge

Projects

None yet

Milestone

17.0

Development

Successfully merging this pull request may close these issues.