Skip to content

[18.0][IMP] rma: Add Create RMA button to rma page#574

Open
victoralmau wants to merge 3 commits intoOCA:18.0from
Tecnativa:18.0-imp-rma-create_rma
Open

[18.0][IMP] rma: Add Create RMA button to rma page#574
victoralmau wants to merge 3 commits intoOCA:18.0from
Tecnativa:18.0-imp-rma-create_rma

Conversation

@victoralmau
Copy link
Copy Markdown
Member

@victoralmau victoralmau commented Apr 13, 2026 •
edited
Loading

Add Create RMA button to rma page

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

@Tecnativa TT61741

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented Apr 13, 2026

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

@pedrobaeza pedrobaeza added this to the 18.0 milestone Apr 13, 2026
@pedrobaeza
Copy link
Copy Markdown
Member

pedrobaeza commented Apr 13, 2026

Can you expand a bit the purpose of the PR?

@victoralmau
Copy link
Copy Markdown
Member Author

victoralmau commented Apr 13, 2026

Can you expand a bit the purpose of the PR?

Use case example:

  • A sales order is created and the delivery picking is done
  • An RMA 1 is created to manage item related to the sales order
  • An RMA 2 is created to manage item related to RMA 1

This wizard is designed to simplify the process of creating RMA 2 directly from RMA 1.

@pedrobaeza
Copy link
Copy Markdown
Member

pedrobaeza commented Apr 13, 2026

I'm afraid I still don't understand RMA from RMA case, but I don't want to block this.

@victoralmau
Copy link
Copy Markdown
Member Author

victoralmau commented Apr 13, 2026

I'm afraid I still don't understand RMA from RMA case, but I don't want to block this.

If a customer (me) purchases something, create a return (RMA 1), and wants to return the item they received, an RMA 2 must be created to process it.

@pedrobaeza
Copy link
Copy Markdown
Member

pedrobaeza commented Apr 13, 2026

Do you mean like a second attempt after the first one was finished?

@victoralmau
Copy link
Copy Markdown
Member Author

victoralmau commented Apr 13, 2026

Do you mean like a second attempt after the first one was finished?

That's right; for example, the product being returned to the customer (in RMA 1) is broken.

@pedrobaeza
Copy link
Copy Markdown
Member

pedrobaeza commented Apr 13, 2026

OK, but at the end, the RMA is linked to the original one. This is only an accelerator for not having to go to the original sales order, isn't it?

@victoralmau
Copy link
Copy Markdown
Member Author

victoralmau commented Apr 13, 2026

OK, but at the end, the RMA is linked to the original one. This is only an accelerator for not having to go to the original sales order, isn't it?

No, this is a new RMA. Since this wizard isn't available, and RMA 1 isn't linked to RMA 2, you need to go to the delivery picking (RMA 1) and use the “Return” button there to create RMA 2.

@victoralmau victoralmau force-pushed the 18.0-imp-rma-create_rma branch 2 times, most recently from 21fc3c2 to d060fe5 Compare April 13, 2026 15:09
@OCA-git-bot OCA-git-bot added mod:rma_delivery Module rma_delivery mod:rma Module rma mod:rma_reason Module rma_reason series:18.0 labels Apr 13, 2026
carlos-lopez-tecnativa
carlos-lopez-tecnativa suggested changes Apr 15, 2026
View reviewed changes
Comment thread rma/views/res_config_settings_views.xml
Comment thread rma/models/res_company.py Outdated
Comment thread rma_delivery/wizard/rma_rma_wizard_views.xml Outdated
Comment thread rma/models/rma.py Outdated
Comment thread rma/views/res_config_settings_views.xml
Comment thread rma/wizard/rma_rma_wizard.py Outdated
Comment thread rma_delivery/wizard/rma_rma_wizard.py Outdated
Comment thread rma/wizard/rma_rma_wizard.py Outdated
@victoralmau victoralmau force-pushed the 18.0-imp-rma-create_rma branch 2 times, most recently from 860fc00 to 4c3baf6 Compare April 15, 2026 12:52
victoralmau added a commit to Tecnativa/rma that referenced this pull request Apr 15, 2026
@victoralmau
[IMP] rma: Change res_config_settings view according to guideline
2327818 
Related to OCA#574 (comment)
victoralmau added a commit to Tecnativa/rma that referenced this pull request Apr 15, 2026
@victoralmau
[IMP] rma_sale: Change res_config_settings view according to guideline
5be853d 
Related to OCA#574 (comment)
@victoralmau victoralmau mentioned this pull request Apr 15, 2026
Open
victoralmau added a commit to Tecnativa/rma that referenced this pull request Apr 15, 2026
@victoralmau
[IMP] rma: Change res_config_settings view according to guideline
883ffad 
Related to OCA#574 (comment)
victoralmau added a commit to Tecnativa/rma that referenced this pull request Apr 15, 2026
@victoralmau
[IMP] rma_sale: Change res_config_settings view according to guideline
876bdc7 
Related to OCA#574 (comment)
@victoralmau
[IMP] rma: Add Create RMA button to rma page
2107e0d 
We want to simplify this process (which can currently be performed by clicking the "Return" button on the delivery picking) and make it possible to carry it out from the RMA page

TT61741
@victoralmau
[IMP] rma_reason: Add fields to rma wizard
4cebd17 
TT61741
@victoralmau
[IMP] rma_delivery: Add fields to rma wizard
941c3b8 
TT61741
@victoralmau victoralmau force-pushed the 18.0-imp-rma-create_rma branch from 4c3baf6 to 941c3b8 Compare April 16, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@carlos-lopez-tecnativa carlos-lopez-tecnativa Awaiting requested review from carlos-lopez-tecnativa

Assignees

No one assigned

Labels

mod:rma_delivery Module rma_delivery mod:rma_reason Module rma_reason mod:rma Module rma series:18.0

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.

4 participants

@victoralmau @OCA-git-bot @pedrobaeza @carlos-lopez-tecnativa