Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16] rma_sale : Always create rma line from sale line using stock move lines #510

Closed

Conversation

florian-dacosta
Copy link
Contributor

This way, we have the same way to create the rma lines, wether the tracking is active on the product or not.
Also and mainly, it allows to create rma lines for the component of a KIT sale order line instead of the kit.
When something is sent as a kit, the rma concerns usually only a part of it.

Maybe I could/should make it an option in the module rma_sale_mrp, so one could choose if the add from sale wizard create a line for the kit of for the components. There is a small hook _create_from_move_line which can be used for this purpose.

The main consequences of this PR then :

  • In case of kit used in sale order line, the "add from sale" wizard will generate one rma line per components (instead of one rma line for the kit)
  • For a normal (not kit) product, with no tracking management, the quantity of the rma line will come from the done stock move line instead of the sale line (same way is is already for the normal (no kit) product with tracking management.

Please, let me know what you think @JordiBForgeFlow @AaronHForgeFlow

@AaronHForgeFlow
Copy link
Contributor

I think both main consequences makes sense, the delivered products are the components of the kit not the kit itself.

…ve lines

This allow to manage phantom bom products, by creating rma lines for the components instead of the kit
Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Makes sense

@AaronHForgeFlow
Copy link
Contributor

Hi @florian-dacosta I checked functionally and I think the change is good. I think that adding the option in the rma_sale_mrp module so user can decide if creating rma lines for the kit components or not is necessary.

@florian-dacosta
Copy link
Contributor Author

Closing, in favor of #525

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants