[18.0][ADD] rma_inter_company: New module#958
Conversation
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
3 times, most recently
from
be0f7f3 to
a2dfe6f
Compare
victoralmau
changed the title
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
3 times, most recently
from
8ca0468 to
5b3f0f9
Compare
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
2 times, most recently
from
b6a3804 to
5075173
Compare
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
5 times, most recently
from
49c3db3 to
e0c6ecc
Compare
juancarlosonate-tecnativa
left a comment
juancarlosonate-tecnativa
left a comment
There was a problem hiding this comment.
Following the flow and validating the RMA picking in Company A, the following error appears:
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
2 times, most recently
from
5533ce2 to
db9b3a2
Compare
victoralmau
changed the title
carlos-lopez-tecnativa
left a comment
carlos-lopez-tecnativa
left a comment
There was a problem hiding this comment.
As mentioned internally, with the current settings, where in C1 I configure that this RMA must generate a new RMA in C2, these settings do not allow generating an RMA in C3, for example. In other words, from one company I can generate an RMA to only a single destination company, but if I want to generate an RMA from C1 to C2 or from C1 to C3, it is not possible. I think this intercompany setting is limited.
rma_inter_company/models/rma.py
Outdated
| if purchase_sale_stock_inter_company is used). | ||
| """ # noqa: E501 | ||
| self.ensure_one() | ||
| return self.company_id.intercompany_rma_company_id |
There was a problem hiding this comment.
When I do not set this field and enable the Generate Intercompany RMA field, I get this error.
rma_inter_company/models/rma.py
Outdated
| partner = company.sudo().partner_id | ||
| vals = { | ||
| "company_id": company.id, | ||
| "partner_id": partner.id, |
There was a problem hiding this comment.
Sorry, but I understand that the partner must be the partner of the origin company, not the destination company. I created the RMA in San Francisco with customer X. This should generate a new RMA in the Chicago company, but the partner should be San Francisco, not Chicago. Or did I misunderstand the flow?
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
5 times, most recently
from
5f392bb to
3007dc1
Compare
victoralmau
force-pushed
the
18.0-add-rma_inter_company
branch
from
3007dc1 to
c1bee2d
Compare
3 participants
New module
Please @pedrobaeza and @carlos-lopez-tecnativa can you review it?
@Tecnativa TT57882