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

[MIG] account_avatax_exemption: Migration to 16.0 #482

Open
wants to merge 29 commits into
base: 16.0
Choose a base branch
from

Conversation

kobros-tech
Copy link

No description provided.

atchuthan and others added 29 commits June 12, 2024 15:23
…rtal' object has no attribute '_get_archive_groups'
…del res.partner.exemption from account_avatax_exemption_base to this account_avatax_exemption
In some cases, the tax rules have been entered in Avatax and you want to synchronize them with Odoo. We add the 'Import Tax Rules' feature that allows you to import the missing rules into Odoo.
@AlexPForgeFlow
Copy link
Contributor

@dnplkndll we reopen #375 and we will try to get it merged.

@kobros-tech
Copy link
Author

@dnplkndll we reopen #375 and we will try to get it merged.

we are testing a new PR in sign project based on this module.

do you know how can we use it as dependency in test requirements for our new sign module?

we are using PRs as dependency if we open PR of the same project but now we have two different repositories.

thanks 🙏

@AlexPForgeFlow
Copy link
Contributor

AlexPForgeFlow commented Jan 29, 2025

@kobros-tech I'm not sure if I got the issue, but to resolve dependencies from PR that are not merged, it's included a file called test-requirements.txt to add Prs with the following syntax:

odoo-addon-account_avatax_exemption@git+https://github.com/OCA/account-fiscal-rule.git@refs/pull/375/head#subdirectory=account_avatax_exemption

See example: https://github.com/ForgeFlow/hr-holidays/blob/17.0-mig-hr_holidays_summary_email/test-requirements.txt

This file should be added in PR from sign project

@kobros-tech
Copy link
Author

@kobros-tech I'm not sure if I got the issue, but to resolve dependencies from PR that are not merged, it's included a file called test-requirements.txt to add Prs with the following syntax:

odoo-addon-account_avatax_exemption@git+https://github.com/OCA/account-fiscal-rule.git@refs/pull/375/head#subdirectory=account_avatax_exemption

See example: https://github.com/ForgeFlow/hr-holidays/blob/17.0-mig-hr_holidays_summary_email/test-requirements.txt

This file should be added in PR from sign project

yes, we did the same but I got errors for this one.

although we are used to do that.

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.

9 participants