Skip to content

[E-Documents Core] - Peppol BIS3.0 validation errors (when both the VAT and the invoice amount are zero) #28320

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

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

GMatuleviciute
Copy link

@GMatuleviciute GMatuleviciute commented Mar 6, 2025

This pull request does not have a related issue as it's part of the delivery for development agreed directly with @altotovi @Groenbech96

Implementation

This update resolves an issue with generating the correct PEPPOL file when both the VAT and the invoice amount are zero. It introduces new validations during the posting of sales documents to ensure that the resulting PEPPOL file complies with tax category rules for services:

  • For document lines with a zero VAT tax category, the system checks that the VAT percentage is 0.
  • For document lines with a standard VAT tax category, the system verifies that the VAT percentage is greater than 0.
  • For document lines with an outside VAT tax category, the system ensures that only one VAT breakdown line is created in the PEPPOL file.

Related Base App PR: https://github.com/microsoft/BusinessCentralApps/pull/1291

Fixes #

@GMatuleviciute GMatuleviciute requested review from a team as code owners March 6, 2025 08:18
Copy link
Contributor

github-actions bot commented Mar 6, 2025

Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link.

@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Integration GitHub request for Integration area
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants