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: ensure vatex reason code is in upper case #56

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

barredterra
Copy link
Member

The code list specifies the codes in lowercase, but some of the schematron files check for uppercase codes only. To ensure successful validation, we uppercase all VAT exemption reason codes.

The code list specifies the codes in lowercase, but some of the schematron files check for uppercase codes only. To ensure successful validation, we uppercase all VAT exemption reason codes.
@barredterra
Copy link
Member Author

@Mergifyio backport version-15 version-14

Copy link
Contributor

mergify bot commented Jan 10, 2025

backport version-15 version-14

✅ Backports have been created

@barredterra barredterra merged commit f63d622 into develop Jan 10, 2025
4 checks passed
@barredterra barredterra deleted the upper-vatex-code branch January 10, 2025 19:30
mergify bot pushed a commit that referenced this pull request Jan 10, 2025
mergify bot pushed a commit that referenced this pull request Jan 10, 2025
barredterra added a commit that referenced this pull request Jan 10, 2025
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
barredterra added a commit that referenced this pull request Jan 10, 2025
Co-authored-by: Raffael Meyer <14891507+barredterra@users.noreply.github.com>
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