Skip to content

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

Closed
GMatuleviciute wants to merge 21 commits intomicrosoft:mainfrom
GMatuleviciute:dev/pmi/PEPPOLZeroAmountDocuments
Closed

[E-Documents Core] - Peppol BIS3.0 validation errors (when both the VAT and the invoice amount are zero)#28320
GMatuleviciute wants to merge 21 commits intomicrosoft:mainfrom
GMatuleviciute:dev/pmi/PEPPOLZeroAmountDocuments

Conversation

@GMatuleviciute
Copy link
Copy Markdown

@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 #

Fixes AB#599334

@GMatuleviciute GMatuleviciute requested review from a team as code owners March 6, 2025 08:18
@github-actions
Copy link
Copy Markdown
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
@Groenbech96 Groenbech96 added the processing-PR The PR is currently being reviewed label Sep 2, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 2, 2025

Processing this PR. The branch is now locked 🔒 Please don't push updates unless otherwise agreed.

@Groenbech96
Copy link
Copy Markdown
Contributor

Processed internally.

@Groenbech96 Groenbech96 closed this Sep 3, 2025
@JesperSchulz JesperSchulz added the ships-in-future-update Fix ships in a future update label Mar 30, 2026
@JesperSchulz
Copy link
Copy Markdown
Contributor

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Build ID: .

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 processing-PR The PR is currently being reviewed ships-in-future-update Fix ships in a future update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants