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(anrok): add fields to graphql response on applied taxes #2473

Merged
merged 4 commits into from
Aug 26, 2024

Conversation

annvelents
Copy link
Contributor

Context

On FE to be able to show differently taxes applied on all line items vs on few only within the invoice, we need to specify on the applied_tax if it's applicable on the whole invoice
and to be able to show different translation for the taxes depending on their code, we want to send tax_cod with applied tax as well

Description

  • added tax_code
  • and applied_on_whole_invoice

to Invoice::AppliedTax graphql type

@annvelents annvelents force-pushed the feat-anrok-add-fields-to-invoice-applied-taxes branch from fcbf5c7 to 303c17c Compare August 23, 2024 15:51
@annvelents annvelents force-pushed the feat-anrok-add-fields-to-invoice-applied-taxes branch from 303c17c to b5675fb Compare August 23, 2024 15:53
@annvelents annvelents marked this pull request as ready for review August 26, 2024 08:14
schema.graphql Outdated Show resolved Hide resolved
@annvelents annvelents force-pushed the feat-anrok-add-fields-to-invoice-applied-taxes branch from b599aca to 529d34a Compare August 26, 2024 09:16
@annvelents annvelents merged commit 7ea6240 into main Aug 26, 2024
6 checks passed
@annvelents annvelents deleted the feat-anrok-add-fields-to-invoice-applied-taxes branch August 26, 2024 09:59
lovrocolic pushed a commit that referenced this pull request Aug 26, 2024
## Context

On FE to be able to show differently taxes applied on all line items vs
on few only within the invoice, we need to specify on the applied_tax if
it's applicable on the whole invoice
and to be able to show different translation for the taxes depending on
their code, we want to send tax_cod with applied tax as well

## Description

- added tax_code
- and applied_on_whole_invoice

to Invoice::AppliedTax graphql type
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