Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMP] sale: change the pricelist in the quotation its update prices o… #31535

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@jva-odoo
Copy link
Contributor

jva-odoo commented Mar 1, 2019

@robodoo robodoo added the seen 🙂 label Mar 1, 2019

@jva-odoo jva-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch Mar 1, 2019

@C3POdoo C3POdoo added the RD label Mar 1, 2019

@robodoo robodoo added the CI 🤖 label Mar 1, 2019

addons/sale/models/sale.py Outdated
@@ -197,6 +197,7 @@ def _get_payment_type(self):
string='Transactions', copy=False, readonly=True)
authorized_transaction_ids = fields.Many2many('payment.transaction', compute='_compute_authorized_transaction_ids',
string='Authorized Transactions', copy=False, readonly=True)
is_change_pricelist = fields.Boolean('Is Change Pricelist', help="To show and hide update price list info")

This comment has been minimized.

@sswapnesh

sswapnesh Mar 1, 2019

Contributor

price list --> Pricelist 🤔

addons/sale/models/sale.py Outdated
def update_pricelist(self):
if not self.env.context.get('skip_price_update'):
for line in self.order_line:
line.product_id_change()

This comment has been minimized.

@sswapnesh

sswapnesh Mar 1, 2019

Contributor

Can't we just update price only instead of other fields(Eg, Description,UoM, account)

This comment has been minimized.

@jva-odoo

jva-odoo Mar 11, 2019

Author Contributor

@sswapnesh Yes, right we also update other fields value...so need to change this method.
Thank you.

@jva-odoo jva-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch Mar 14, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 14, 2019

@hdh-odoo hdh-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch Mar 15, 2019

@robodoo robodoo removed the CI 🤖 label Mar 15, 2019

@hdh-odoo hdh-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch Mar 15, 2019

@robodoo robodoo added the CI 🤖 label Mar 15, 2019

@hdh-odoo hdh-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch to 0dac619 Mar 15, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 15, 2019

@mba-odoo mba-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch Mar 15, 2019

@robodoo robodoo added the CI 🤖 label Mar 15, 2019

[IMP] sale: change the pricelist in the quotation its update prices o…
…f product.

purpose
======
changing the pricelist in the quotation doesn't update prices.

specs
=====
When the pricelist is changed with some order lines already set,
throw a modal to suggest the user to update prices.
like Do you want to recompute the existing prices? Yes, No

Related Task ID: 1947320
Closes : #31535

@mba-odoo mba-odoo force-pushed the odoo-dev:master-sale-update-pricelist-jva branch to 93b942f Mar 15, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.