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

[FIX] Avatax: Use Odoo Tax option #449

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

AlexPForgeFlow
Copy link
Contributor

Forward Port from commit: 4c56a80

@OCA-git-bot
Copy link
Contributor

Hi @dreispt,
some modules you are maintaining are being modified, check this out!

@AlexPForgeFlow
Copy link
Contributor Author

Hi @atchuthan I cherry-picked your fix, but I have one question, I hope you can help me.
In your proposal, you use the field avatax_tax_type, to manage visibility for option "Use Odoo Tax". The problem I have is that for those invoices that taxes have already been computes if I want to create a credit note, option "Use Odoo Tax" would not be visible, and I don't want to recompute taxes for those invoices because taxes may have changed. My question is, there is some default code I can set for avatax_tax_type? Wouldn't be the same if instead of using avatax_tax_type for option visibility we use field is_avatax or check avatax_amt_line.

Thanks!

@atchuthan
Copy link
Member

Hi @atchuthan I cherry-picked your fix, but I have one question, I hope you can help me. In your proposal, you use the field avatax_tax_type, to manage visibility for option "Use Odoo Tax". The problem I have is that for those invoices that taxes have already been computes if I want to create a credit note, option "Use Odoo Tax" would not be visible, and I don't want to recompute taxes for those invoices because taxes may have changed. My question is, there is some default code I can set for avatax_tax_type? Wouldn't be the same if instead of using avatax_tax_type for option visibility we use field is_avatax or check avatax_amt_line.

Thanks!

For old records of our customer, we didn't have the tax type, so we had to make code change to check if avatax_tax_type is set in order to work with "Use Odoo Tax". I think we can change as per your suggestion and generalize them.

Thanks

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.

3 participants