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

[IMP] l10n_mx: Import taxes are supported now #328

Closed

Conversation

luistorresm
Copy link
Collaborator

User Case:
The Company A buy some products from a Foreign Customer, and when they
are received in Mexico, the Broker Partner send an invoice to Company A
by the importation taxes.

In this case, when Company A, make the payment to the broker, the taxes
could be accredited for the company and in your case must be reported in
the DIOT report.

More info about the configuration in the README file.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

User Case:
The Company A buy some products from a Foreign Customer, and when they
are received in Mexico, the Broker Partner send an invoice to Company A
by the importation taxes.

In this case, when Company A, make the payment to the broker, the taxes
could be accredited for the company and in your case must be reported in
the DIOT report.

More info about the configuration in the README file.
@luistorresm
Copy link
Collaborator Author

Hello @hbto

Do you have news about this PR?

Regards

@hbto
Copy link

hbto commented Oct 2, 2019

@luistorresm you added a test but do not actually tested that the invoice would have created the journal items according what is needed to report DIOT.

Regards.

@luistorresm
Copy link
Collaborator Author

@hbto

This module (l10n_mx) not have the DIOT report, the test could be added in l10n_mx_enterprise.

What do you think?

Regards

@hbto
Copy link

hbto commented Oct 2, 2019

👍

@hbto
Copy link

hbto commented Oct 2, 2019

Approved for merge

@luistorresm
Copy link
Collaborator Author

Odoo PR odoo#37815

@luistorresm luistorresm closed this Oct 2, 2019
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.

2 participants