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

[17.0][MIG] rma_repair #570

Open
wants to merge 47 commits into
base: 17.0
Choose a base branch
from
Open

[17.0][MIG] rma_repair #570

wants to merge 47 commits into from

Conversation

JasminSForgeFlow
Copy link
Collaborator

@JasminSForgeFlow JasminSForgeFlow commented Jan 8, 2025

Adapt changes of odoo/odoo@eaa6e1a

JordiBForgeFlow and others added 30 commits January 7, 2025 12:35
* fix quantity computation
* use a payment_state field instead of inovice state one.
* Use proper digits and unit conversion.
* Add a filter "To repair".
* Remove dead code and clean warnings.
The issue odoo/odoo#68370 is affecting
the test of rma_repair, commenting them while the issue is
attended.
editable in draft state in the related RMA, therefore when you
have created a repair order, the field `under_warranty` is not
editable anymore.
This reverts commit 6825ce6.
The issue has been attended and fixed.
right in v14. Actions that read those records need to use
`sudo` to allow non-admin users to be able to use these actions.
MateuGForgeFlow and others added 16 commits January 7, 2025 12:35
…srbish

[IMP] rma_repair: cosmetics changes due to copier update
OCA was added in the past by mistake only because a misconfiguration
in the CI forced OCA as a required author.
When not using rma_put_away you can define push rules anyway to move the products
to the repair location, and those transfers count as rma repair transfers

if you install and use rma_put_away then the rma repair transfers are the ones
from the putaway moves

Before this commit there is a dependency issue because the rma_repair code
is using a field declared in the rma_put_away, this commit fixes that issue

[IMP] rma_repair_put_away: added tests
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.

10 participants