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

[ADD] l10n_ar_edi_ux: compatibility with module subchange #179

Closed
wants to merge 1 commit into from

Conversation

zaoral
Copy link
Contributor

@zaoral zaoral commented Feb 7, 2023

Be able to auto complete the missing required EDI values when the subchange is automatically created because of the use of financial plan

@roboadhoc
Copy link
Contributor

Be able to auto complete the missing required EDI values when the
subchange is automatically created because of the use of financial plan
@zaoral
Copy link
Contributor Author

zaoral commented Feb 10, 2023

@roboadhoc r+

@roboadhoc
Copy link
Contributor

I'm sorry, @zaoral: you can't review+.

@ica-adhoc
Copy link
Contributor

@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Feb 10, 2023
Be able to auto complete the missing required EDI values when the
subchange is automatically created because of the use of financial plan

closes #179

Related: ingadhoc/account-payment#349
Signed-off-by: Ignacio Cainelli <ica@adhoc.com.ar>
roboadhoc pushed a commit to ingadhoc/account-payment that referenced this pull request Feb 10, 2023
If the financial plan does not add any subcharge we should not
create a ND amount 0. Only create the ND when we have a subcharge to
apply

closes #349

Related: ingadhoc/odoo-argentina-ee#179
Signed-off-by: Ignacio Cainelli <ica@adhoc.com.ar>
@roboadhoc roboadhoc temporarily deployed to merge February 10, 2023 20:32 Inactive
@roboadhoc roboadhoc closed this Feb 10, 2023
@fw-bot-adhoc fw-bot-adhoc deleted the 13.0-h-59089-kz branch February 24, 2023 21:31
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.

None yet

3 participants