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

[FIX] website_event,website_event_sale: correction pricelist in event… #31953

Conversation

Projects
None yet
5 participants
@alt-odoo
Copy link
Contributor

commented Mar 19, 2019

… register

A recent fix c41d1d2 was made to select the related pricelist
to show the correct price of an event product.

This fix creates another bug in case of Ecommerce app
not installed, because the method get_current_pricelist
is defined in website_sale module.

We need to move the fix from website_event to website_event_sale module.

Description of the issue/feature this PR addresses:

OPW-1957358

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

[FIX] website_event,website_event_sale: correction pricelist in event…
… register

A recent fix c41d1d2 was made to select the related pricelist
to show the correct price of an event product.

This fix creates another bug in case of Ecommerce app
not installed, because the method get_current_pricelist
is defined in website_sale module.

We need to move the fix from website_event to website_event_sale module.
@nle-odoo

This comment has been minimized.

Copy link
Contributor

commented Mar 20, 2019

seems good

robodoo delegate=jpp-odoo

@jpp-odoo

This comment has been minimized.

Copy link
Contributor

commented Mar 20, 2019

robodoo r+

@robodoo robodoo added the r+ 👌 label Mar 20, 2019

@robodoo robodoo closed this in f7acbf6 Mar 20, 2019

@robodoo robodoo added merged 🎉 and removed merging 👷 labels Mar 20, 2019

@robodoo

This comment has been minimized.

Copy link
Contributor

commented Mar 20, 2019

Merged, thanks!

@nle-odoo nle-odoo deleted the odoo-dev:12.0-get-current-pricelist-website-event-sale-alt branch Mar 20, 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.