Skip to content

[18.0][IMP] rma + rma_sale: Add new rma compatibility#591

Open
victoralmau wants to merge 2 commits into
OCA:18.0from
Tecnativa:18.0-imp-rma_sale-TT62775
Open

[18.0][IMP] rma + rma_sale: Add new rma compatibility#591
victoralmau wants to merge 2 commits into
OCA:18.0from
Tecnativa:18.0-imp-rma_sale-TT62775

Conversation

@victoralmau
Copy link
Copy Markdown
Member

@victoralmau victoralmau commented Jun 1, 2026

Add new rma compatibility

Use case example:

  • Create a sales order
  • Confirm the sales order + done picking
  • Create RMA 1 from the sales order
  • Done reception picking of RMA 1 + return to the customer + done delivery picking for RMA 1
  • Create RMA 2 from RMA 1
  • Done reception picking of RMA 2
  • Refund RMA 2
  • Credit note must be linked to the sales order

Please @pedrobaeza and @carlos-lopez-tecnativa can you review it?

@Tecnativa TT62775

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @chienandalu, @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added mod:rma Module rma series:18.0 mod:rma_sale Module rma_sale labels Jun 1, 2026
@pedrobaeza pedrobaeza added this to the 18.0 milestone Jun 1, 2026
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Isn't this one fix and one imp that are not related (so 2 commits)?

…_create_rma() method

An error will be displayed if the action_create_rma() method is called manually and it is not possible to do so

TT62775
Use case example:
- Create a sales order
- Confirm the sales order + done picking
- Create RMA 1 from the sales order
- Done reception picking of RMA 1 + return to the customer + done delivery picking for RMA 1
- Create RMA 2 from RMA 1
- Done reception picking of RMA 2
- Refund RMA 2
- Credit note must be linked to the sales order

TT62775
@victoralmau victoralmau force-pushed the 18.0-imp-rma_sale-TT62775 branch from 8a64284 to ee4c937 Compare June 1, 2026 10:02
Copy link
Copy Markdown

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants