Skip to content

[18.0][IMP] rma_lot: Add domain_lot_id field to lot_id field#586

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-rma_lot-domain_lot_id
Jun 1, 2026
Merged

[18.0][IMP] rma_lot: Add domain_lot_id field to lot_id field#586
OCA-git-bot merged 1 commit into
OCA:18.0from
Tecnativa:18.0-imp-rma_lot-domain_lot_id

Conversation

@victoralmau
Copy link
Copy Markdown
Member

@victoralmau victoralmau commented May 19, 2026

Add domain_lot_id field to lot_id field

Similar to sale_order_lot_selection

Please @pedrobaeza and @pilarvargas-tecnativa can you review it?

@Tecnativa TT62586

@pedrobaeza pedrobaeza added this to the 18.0 milestone May 19, 2026
Comment thread rma_lot/models/rma.py
Similar to sale_order_lot_selection

TT61756
@victoralmau victoralmau force-pushed the 18.0-imp-rma_lot-domain_lot_id branch from 6445fac to a6d584e Compare May 19, 2026 11:56
@victoralmau
Copy link
Copy Markdown
Member Author

This is ready to merge, right?

@pedrobaeza
Copy link
Copy Markdown
Member

Not really, as the rules state that it should be 3 approvals or 2 and 5 working days, and this one not being our module, it's not fair to not let others to give their opinion being a new behavior. If we have the @sbejaoui's blessing, we can continue.

@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). 🤖

@victoralmau
Copy link
Copy Markdown
Member Author

Ping @carlos-lopez-tecnativa

Copy link
Copy Markdown
Contributor

@sbejaoui sbejaoui left a comment

Choose a reason for hiding this comment

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

LGTM, thank you

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge minor

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-586-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit c7e7d88 into OCA:18.0 Jun 1, 2026
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-imp-rma_lot-domain_lot_id branch June 1, 2026 11:44
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.

5 participants