[18.0][FIX] rma + rma_sale: Full RMA workflow without additional permissions (Inventory or Sales)#583
Conversation
|
Hi @pedrobaeza, @chienandalu, |
|
To which point is the RMA module functional without stock permissions? Should we go this way to restrict visibility, or should we grant equivalent stock permissions to RMA users? |
|
Although it’s not common (or at least that’s what I think) to use the RMA module without inventory permissions, it could happen; I don’t think inheriting permissions is appropriate, nor is adding ACLs for everything related to inventory, which is why these changes have been done. There are some changes related to sales permissions that I could probably add in a separate commit or a separate PR if you prefer. |
… have permissions in Inventory
…s not have permissions in Sales
It is important that all delivery moves be either canceled or done, because a delivery move might be created, canceled (for example, because the wrong product was used), recreated, and then done.
victoralmau
force-pushed
the
18.0-fix-rma-rma-user
branch
from
b9c3826 to
9c5f3fb
Compare
|
/ocabot merge minor |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 76a7366. Thanks a lot for contributing to OCA. ❤️ |
4 participants
Full RMA workflow without additional permissions (Inventory or Sales)
Please @pedrobaeza and @carlos-lopez-tecnativa can you review it?
@Tecnativa