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_account #559

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

[18.0][MIG] rma_account #559

wants to merge 103 commits into from

Conversation

JasminSForgeFlow
Copy link
Collaborator

Depends on:

JordiBForgeFlow and others added 30 commits November 27, 2024 11:06
* 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 attributes.
* simplify action_view methods.
* fix wrong naming.
* fix misplaced views.
* fix wrong count and view actions for rma.orders in invoices.
* fix error when installing the module.
* remove unneded data update when preparing rma lines from invoice lines.
* minor extra fixes.
 * fix view
 * small issue in wizard
 * modify data file
 * smaller issues
DavidJForgeFlow and others added 25 commits November 27, 2024 11:06
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.
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.
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.
when typing the complete invoice name
In v16, fiscal_position_id is a computed field, so if we set up the field in the refund creation we are overriding the computed value
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.