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

[18.0][MIG] rma_purchase #560

Open
wants to merge 69 commits into
base: 18.0
Choose a base branch
from
Open

Conversation

JasminSForgeFlow
Copy link
Collaborator

JordiBForgeFlow and others added 30 commits November 28, 2024 09:35
* fix assignment of moves.
* default qty in rma lines.
* remove account dependency.
* test and flake8 fixes.
… supplier

[IMP]Separate menus for customer and supplier operations
* Add active field to rma operation
* Added tests
* Fix travis
* Fix create supplier rma from customer rma
* remove unneded copy and ondelete attributes.
* simplify action_view methods.
* fix rma line supplier view.
* fix wizard.
* extend README.
* minor extra fixes.
Conflicts:
	rma_purchase/models/purchase_order_line.py
This reverts commit dfa54da.
MiquelRForgeFlow and others added 30 commits November 28, 2024 09:35
right in v14. Actions that read those records need to use
`sudo` to allow non-admin users to be able to use these actions.
…rrect price unit in moves

Otherwise the inventory accounting will be completely wrong.
[FIX] rma: rma_custmer_user has no write permissions in partner, so compute method fails.
[IMP] rma: use rma user in tests
[FIX] rma_account: move_line_id field string
[IMP] rma, rma_account, rma_sale, rma_purchase: tests for stock valuation
[FIX] account_move_line_rma_order_line: minor lint, make auto-install
OCA was added in the past by mistake only because a misconfiguration
in the CI forced OCA as a required author.
Otherwise the anglo saxon entries won't be correct.
For example, the Interim (Delivered) account should balance
after receiving and triggering a refund on a customer rma.
"line" is an rma.order.line while purchase_line_id is a relation
to purchase.order.line.
…lied

Without this, some policies are not being copied from the
operation selected when creating new rma line from a rma group.

In v16 this patch and the usage of such onchange can be removed
in favor of (pre)computed stored editable fields for all policies
and configuration in the RMA operation.
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.