odoo-shopinvader icon indicating copy to clipboard operation
odoo-shopinvader copied to clipboard

[16.0][ADD] shopinvader_search_engine_product_multi_price

Open qgroulard opened this issue 1 year ago • 6 comments

See https://github.com/shopinvader/odoo-shopinvader/pull/1441 and https://github.com/shopinvader/odoo-shopinvader/pull/1467 for PRs on related matter.

  1. Remove price from the schema in shopinvader_product as there are a lot of different ways to expose product prices and they should be handled in dedicated modules. '_get_price()' method has already been extracted to OCA/product-attribute module 'product_get_price_helper'.

  2. Add a module to export prices for a list of pricelists set on the index or backend.

qgroulard avatar Jan 26 '24 15:01 qgroulard

@qgroulard the 1st commit can be dropped now, no?

simahawk avatar Feb 26 '24 08:02 simahawk

@qgroulard the 1st commit can be dropped now, no?

In theory yes, the keys price and prices can coexist.

But I think price should be removed anyway as it is a broken first draft. Prices were still WIP when schemas have been merged, keeping it on the main branch seems wrong to me.

qgroulard avatar Feb 26 '24 09:02 qgroulard

@qgroulard the 1st commit can be dropped now, no?

In theory yes, the keys price and prices can coexist.

But I think price should be removed anyway as it is a broken first draft. Prices were still WIP when schemas have been merged, keeping it on the main branch seems wrong to me.

I meant that you already removed it when creating the helper module, no?

simahawk avatar Feb 26 '24 09:02 simahawk

@qgroulard the 1st commit can be dropped now, no?

In theory yes, the keys price and prices can coexist. But I think price should be removed anyway as it is a broken first draft. Prices were still WIP when schemas have been merged, keeping it on the main branch seems wrong to me.

I meant that you already removed it when creating the helper module, no?

It seems not, in fact :sweat_smile:

simahawk avatar Feb 26 '24 10:02 simahawk

@qgroulard can we fast track https://github.com/shopinvader/odoo-shopinvader/pull/1507 while fixing this one?

simahawk avatar Feb 26 '24 11:02 simahawk

@qgroulard ping :)

simahawk avatar Apr 08 '24 07:04 simahawk

/ocabot merge nobump

lmignon avatar Jun 04 '24 16:06 lmignon

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

shopinvader-git-bot avatar Jun 04 '24 16:06 shopinvader-git-bot

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

shopinvader-git-bot avatar Jun 04 '24 16:06 shopinvader-git-bot