OCA / rma

Odoo for Return Merchandise Authorization (RMA)
GNU Affero General Public License v3.0
78 stars 206 forks source link

RMA reason #405

Open jbaudoux opened 3 months ago

jbaudoux commented 3 months ago

Add rma reason

To improve quality, it is good to understand why an RMA has been created.

For this I'd like to have a required RMA reason defined on the RMA. This would be also available and required on the SO wizard on each line (or globally)

Make a rma_reason module that depends on rma

Make a rma_sale_reason module that depends on rma_sale and rma_reason

chienandalu commented 3 months ago

Well, that's what the RMA operations are for. The module pre-creates three of them, but you can change them or add as many as you want to.

And customers in the sale portal can select them and even add notes per return line

jbaudoux commented 3 months ago

In fact, it serves two different purpose. I use the operation to dictate what process should apply (return, refund, replace...) and which will create the right transfer (incoming, delivery or both, credit note...). In my use cases, it is the after-sales service that choose the right operation to apply. The reason would explain the cause of this RMA like: wrong item delivered, customer don't want the product anymore, product has a defect, wrong product description... In the same spirit than https://github.com/OCA/sale-workflow/tree/16.0/sale_cancel_reason

chienandalu commented 3 months ago

I understand. There's a similar optional feature (group_rma_manual_finalization) for manual finalization reasons. Maybe you could propose a similar approach.

jbaudoux commented 3 months ago

I understand. There's a similar optional feature (group_rma_manual_finalization) for manual finalization reasons. Maybe you could propose a similar approach.

Can you provide the link to that module, I can't find it ?

chienandalu commented 3 months ago

It's in the rma module itself. Follow the code for this option:

https://github.com/OCA/rma/blob/eeb06725c0b79d5777680f194a1edf600f74e069/rma/models/res_config_settings.py#L9-L13

jbaudoux commented 2 months ago

I updated the issue with the proposed solution specifications