Skip to content

Peppol App fixes #6268

Merged
Groenbech96 merged 17 commits intomainfrom
copilot/overwrite-get-accounting-supplier-tax-scheme
Feb 16, 2026
Merged

Peppol App fixes #6268
Groenbech96 merged 17 commits intomainfrom
copilot/overwrite-get-accounting-supplier-tax-scheme

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026

Updated to Peppol App. Uptakes new method in Baseapp, allows us to create smaller BE specific app in NAV. It can reuse method in baseapp that is localized.

Fixes AB#621947

Copilot AI changed the title [WIP] Override GetAccountingSupplierPartyTaxScheme method for Peppol app Add BE Peppol app with Belgium-specific VAT registration formatting Jan 23, 2026
Copilot AI requested a review from Groenbech96 January 23, 2026 16:55
Copilot AI changed the title Add BE Peppol app with Belgium-specific VAT registration formatting Add BE Peppol app with Enterprise No. validation and VAT formatting Jan 23, 2026
Copilot AI changed the title Add BE Peppol app with Enterprise No. validation and VAT formatting Add BE Peppol localization with Enterprise No. support Jan 23, 2026
@Groenbech96
Copy link
Contributor

@copilot try again

@Groenbech96
Copy link
Contributor

@copilot try again

@Groenbech96
Copy link
Contributor

@copilot try again

…50, add InternalsVisibleTo

Co-authored-by: Groenbech96 <[email protected]>
Copy link
Contributor Author

Copilot AI commented Jan 30, 2026

@copilot Please implement Arthurs comments.

Implemented all of Arthur's comments in cd8a2e5:

  • Added Access = Internal to PEPPOL30BESalesValidation and PEPPOL30BEServiceValidation codeunits
  • Lowered ID range from 37400 to 37350
  • Added InternalsVisibleTo from W1 PEPPOL app to BE app

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.


🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

Magnus Hartvig Grønbech added 3 commits February 10, 2026 15:02
@github-actions github-actions bot added this to the Version 28.0 milestone Feb 11, 2026
ventselartur
ventselartur previously approved these changes Feb 11, 2026
@Groenbech96 Groenbech96 changed the title Add BE Peppol localization with Enterprise No. support Peppol App fixes Feb 11, 2026
@Groenbech96
Copy link
Contributor

@copilot pull main into the branch.

Copy link
Contributor Author

Copilot AI commented Feb 16, 2026

@copilot pull main into the branch.

I'm unable to pull main into this branch. My environment doesn't allow me to fetch, pull, or merge branches from GitHub. You'll need to run git pull origin main or merge main into this branch manually.

…ot/overwrite-get-accounting-supplier-tax-scheme
@Groenbech96 Groenbech96 added Approved The issue is approved Linked Issue is linked to a Azure Boards work item labels Feb 16, 2026
@Groenbech96 Groenbech96 enabled auto-merge (squash) February 16, 2026 15:29
@Groenbech96 Groenbech96 merged commit 398882d into main Feb 16, 2026
67 of 70 checks passed
@Groenbech96 Groenbech96 deleted the copilot/overwrite-get-accounting-supplier-tax-scheme branch February 16, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AL: Apps (W1) Add-on apps for W1 Approved The issue is approved Linked Issue is linked to a Azure Boards work item

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants