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

feat: #755 taxes on payments #781

Closed
wants to merge 1 commit into from

Conversation

mildred
Copy link
Contributor

@mildred mildred commented Dec 2, 2023

When defining taxes, it is possible to define an additional payment account that will be used during payments to move taxes from the original tax account to this new payment tax account. This allows to account for taxes only when payment is received.

Now payments can reference tax summary objects that will reference the two accounts to move funds between when the payment is committed. Reuse some of the Invoice code to generate these tax summary objects.

@mildred mildred self-assigned this Dec 2, 2023
@mildred mildred force-pushed the 755-taxes-on-payments branch 2 times, most recently from d12c74e to b0ec86d Compare December 5, 2023 20:41
@mildred mildred changed the title #755 taxes on payments feat: #755 taxes on payments Dec 5, 2023
@mildred mildred force-pushed the 755-taxes-on-payments branch from b0ec86d to 1de83ae Compare December 5, 2023 20:43
@mildred mildred force-pushed the 755-taxes-on-payments branch from 1de83ae to a93fdd7 Compare December 21, 2023 20:05
When defining taxes, it is possible to define an additional payment
account that will be used during payments to move taxes from the
original tax account to this new payment tax account. This allows to
account for taxes only when payment is received.

Now payments can reference tax summary objects that will reference the
two accounts to move funds between when the payment is committed. Reuse
some of the Invoice code to generate these tax summary objects.
@mildred mildred force-pushed the 755-taxes-on-payments branch from a93fdd7 to b4041f5 Compare December 21, 2023 20:19
@mildred mildred mentioned this pull request Dec 22, 2023
@mildred
Copy link
Contributor Author

mildred commented Dec 22, 2023

replaced by #789 (same code different branch)

@mildred mildred closed this Dec 22, 2023
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.

1 participant