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] account_avatax_exemption: Migration to 18.0 #475

Open
wants to merge 34 commits into
base: 18.0
Choose a base branch
from

Conversation

dnplkndll
Copy link
Contributor

@dnplkndll dnplkndll commented Jan 15, 2025

fix: model_create_multi

fix: _search()
@dnplkndll
Copy link
Contributor Author

@atchuthan can you look over? I tried to fix to get any commits from you guys in from 17. testing my git skills though and overwrote it once!

@dnplkndll dnplkndll force-pushed the 18.0-mig-account_avatax_exemption branch from 9667481 to f0f25b5 Compare January 15, 2025 21:26
@dnplkndll dnplkndll force-pushed the 18.0-mig-account_avatax_exemption branch from f0f25b5 to 6c9a27d Compare January 15, 2025 21:34
@kobros-tech
Copy link

kobros-tech commented Feb 6, 2025

@dnplkndll
@wlin-kencove

Based on #473 the form view should look like:

Screenshot from 2025-02-06 02-40-51

Screenshot from 2025-02-06 03-32-03

Screenshot from 2025-02-06 02-41-03

@kobros-tech kobros-tech force-pushed the 18.0-mig-account_avatax_exemption branch 2 times, most recently from ee0fdfe to 3e462df Compare February 6, 2025 14:52
@kobros-tech kobros-tech force-pushed the 18.0-mig-account_avatax_exemption branch from 3e462df to 6ab1aa5 Compare February 6, 2025 15:33
@dnplkndll
Copy link
Contributor Author

@benstannard since we introduce the exemption rules here. should we detail on the description the rules around SST compliance? don't want to have anyone get into trouble by misapplying those if participating in the program.
I wonder if useful to ad a general SST compliance check would be useful. since you would need the members enabled? that would be more applicable on the base tax system module for states and I guess the new nexus check is not in odoo code now?

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.

8 participants