Skip to content

[18.0][MIG] sale_order_lot_selection: Migration to 18.0#3409

Merged
OCA-git-bot merged 70 commits intoOCA:18.0from
adhoc-dev:18.0-mig-sale_order_lot_selection
Mar 29, 2025
Merged

[18.0][MIG] sale_order_lot_selection: Migration to 18.0#3409
OCA-git-bot merged 70 commits intoOCA:18.0from
adhoc-dev:18.0-mig-sale_order_lot_selection

Conversation

@lef-adhoc
Copy link
Copy Markdown
Contributor

@lef-adhoc lef-adhoc commented Nov 14, 2024

@lef-adhoc lef-adhoc changed the title [18.0 mig sale order lot selection [18.0][MIG] sale_order_lot_selection: Migration to 18.0 Nov 14, 2024
@lef-adhoc lef-adhoc force-pushed the 18.0-mig-sale_order_lot_selection branch 2 times, most recently from fa1c869 to 1cd6283 Compare November 14, 2024 18:04
@pedrobaeza
Copy link
Copy Markdown
Member

Everything is correct. The red CI is there for avoiding to merge this while the dependency is not merged.

/ocabot migration sale_order_lot_selection

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 14, 2024
@Abimael1321
Copy link
Copy Markdown

Functionally reviewed. LGTM!

hurrinico and others added 22 commits March 6, 2025 11:17
[FIX] Fix product_id on test

[FIX] Fix product on test
[FIX] Use product_14 instead of product_11

[FIX] Add test's flow step
[FIX] change data in quant search

[FIX] Fixed tab

[FIX] Add check on lots list

FIX product_id_change_with_wh
Because quants_get_prefered_domain does not return quants with our lots, if other quants are available
@matiasperalta1 matiasperalta1 force-pushed the 18.0-mig-sale_order_lot_selection branch from 1cd6283 to 12d20d4 Compare March 6, 2025 14:18
@matiasperalta1
Copy link
Copy Markdown
Contributor

@pedrobaeza Everything seems ok. Could you merge it?

@pedrobaeza
Copy link
Copy Markdown
Member

A PSC / maintainer should review it.

# © 2015 Agile Business Group
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).

from odoo.tests import TransactionCase
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
from odoo.tests import TransactionCase
from odoo.addons.base.tests.common import BaseCommon

It is recommended to use BaseCommon.

from odoo.tests import TransactionCase


class TestSaleOrderLotSelection(TransactionCase):
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
class TestSaleOrderLotSelection(TransactionCase):
class TestSaleOrderLotSelection(BaseCommon):

Copy link
Copy Markdown

@rjaraspearhead rjaraspearhead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGM 👍

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@AEstLo
Copy link
Copy Markdown

AEstLo commented Mar 28, 2025

@bodedra do you think this can be merged? (as maintainer https://github.com/OCA/sale-workflow/tree/17.0/sale_order_lot_selection#maintainers)

@bodedra
Copy link
Copy Markdown
Member

bodedra commented Mar 29, 2025

Functional reviewed. 👍

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 6b22fdd into OCA:18.0 Mar 29, 2025
2 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

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.