OCA / product-configurator

Advanced Product Configurator (from the original Pledra project https://github.com/pledra/odoo-product-configurator)
GNU Affero General Public License v3.0
33 stars 78 forks source link

[FIX] product_configurator_sale: Price unit comes from the session #119

Closed SirAionTech closed 1 month ago

SirAionTech commented 8 months ago

When changing the configuration session of an order line, the price unit should update accordingly.

The code dedicated to onchanges was probably needed in 14.0 due to https://github.com/odoo/odoo/blob/c0acceab75c23460c7d9ad6cafcd65ce795cea9f/addons/sale/models/sale.py#L1293 but in 16.0 such code does not exist anymore and onchanges are used very little.

OCA-git-bot commented 8 months ago

Hi @PCatinean, some modules you are maintaining are being modified, check this out!

github-actions[bot] commented 4 months ago

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

SirAionTech commented 4 months ago

Rebased on 16.0

simahawk commented 1 month ago

LG, just code review

simahawk commented 1 month ago

/ocabot merge patch

OCA-git-bot commented 1 month ago

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

OCA-git-bot commented 1 month ago

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