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
ChrisOForgeFlow and others added 24 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
@DavidJForgeFlow
Copy link
Contributor

DavidJForgeFlow commented Feb 19, 2025

Hi @JasminSForgeFlow could you please add changes in #580 and #583?
4d94bf8c44c76ca608cfe48092afaa6207cee485
4bf3f6ecfa1fda122131bb31e8c00c5bea0cc1c3
4a518c3a4288fbfe6c400df2633e48554f356994

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.