Skip to content

[IMP] product_catalog_tree: add button to load vendor min qty in list catalog - #915

Closed
fw-bot-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-18.0-t-121116-jc-6757-fw
Closed

[IMP] product_catalog_tree: add button to load vendor min qty in list catalog#915
fw-bot-adhoc wants to merge 1 commit into
ingadhoc:19.0from
adhoc-dev:19.0-18.0-t-121116-jc-6757-fw

Conversation

@fw-bot-adhoc

Copy link
Copy Markdown

Problem

In the list-view catalog the + button (increase_quantity) always adds 1
unit. The kanban catalog instead suggests the vendor's min_qty on the
first add (see purchase product_catalog/kanban_record.jsaddProduct).

As a consequence, adding a product whose vendor min_qty is greater than the
added quantity from the list catalog selects no seller, and the price falls
back to the product cost — while doing the same from the kanban catalog applies
the vendor price. Same product, different price depending on the view used.

Change

  • Expose product_catalog_min_qty on the catalog row (taken from the catalog
    payload min_qty).
  • Add a dedicated button add_catalog_min_qty that mirrors the kanban
    behaviour: on the first add it uses the vendor min_qty (so the vendor price
    applies instead of the cost); once the line already exists it adds one, like
    the + button. The button is only shown when the vendor min_qty is greater
    than 1.
  • Add tests (skipped when purchase is not installed).

The existing + button keeps its behaviour unchanged.

Forward-Port-Of: #908

@roboadhoc

Copy link
Copy Markdown
Contributor

Pull request status dashboard

@fw-bot-adhoc

Copy link
Copy Markdown
Author

@jcadhoc @les-adhoc cherrypicking of pull request #908 failed.

stdout:

Auto-merging product_catalog_tree/__manifest__.py
CONFLICT (content): Merge conflict in product_catalog_tree/__manifest__.py
Auto-merging product_catalog_tree/models/product_product.py
Auto-merging product_catalog_tree/views/product_product_views.xml

Either perform the forward-port manually (and push to this branch, proceeding as usual) or close this PR (maybe?).

:shipit: you can use git-fw to re-do the forward-port for you locally.

⚠️ after resolving this conflict, you will need to merge it via @roboadhoc.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@fw-bot-adhoc

Copy link
Copy Markdown
Author

@jcadhoc @les-adhoc this forward port of #908 is awaiting action (not merged or closed).

2 similar comments
@fw-bot-adhoc

Copy link
Copy Markdown
Author

@jcadhoc @les-adhoc this forward port of #908 is awaiting action (not merged or closed).

@fw-bot-adhoc

Copy link
Copy Markdown
Author

@jcadhoc @les-adhoc this forward port of #908 is awaiting action (not merged or closed).

@jcadhoc
jcadhoc force-pushed the 19.0-18.0-t-121116-jc-6757-fw branch from 4167813 to 9079b8d Compare August 10, 2026 13:48
@les-adhoc

Copy link
Copy Markdown
Contributor

@roboadhoc r+ nobump

@roboadhoc

Copy link
Copy Markdown
Contributor

@jcadhoc @les-adhoc unable to stage: All commits must have author and committer email, missing email on 9079b8d indicates the authorship is most likely incorrect.

… catalog

Forward-port of ingadhoc#908 plus its follow-up d79511e: the
truck button sets the vendor min_qty (so the vendor price applies)
instead of adding one like the '+' button.

Resolved forward-port conflict in __manifest__.py (version -> 19.0.1.4.0).

Forward-Port-Of: ingadhoc#908
@jcadhoc
jcadhoc force-pushed the 19.0-18.0-t-121116-jc-6757-fw branch from 9079b8d to 4ee02eb Compare August 10, 2026 15:14
@fw-bot-adhoc

Copy link
Copy Markdown
Author

@jcadhoc @les-adhoc this forward port of #908 is awaiting action (not merged or closed).

@les-adhoc

Copy link
Copy Markdown
Contributor

@roboadhoc r+ nobump

@roboadhoc roboadhoc closed this in 5523073 Aug 21, 2026
@roboadhoc
roboadhoc deleted the 19.0-18.0-t-121116-jc-6757-fw branch August 21, 2026 14:07
@roboadhoc roboadhoc added the 18.1 label Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants