Skip to content

[E-Documents Core] - Internal visibility for Digital Vouchers#8677

Open
GMatuleviciute wants to merge 6 commits into
microsoft:mainfrom
GMatuleviciute:dev/aan/digitalvoucher
Open

[E-Documents Core] - Internal visibility for Digital Vouchers#8677
GMatuleviciute wants to merge 6 commits into
microsoft:mainfrom
GMatuleviciute:dev/aan/digitalvoucher

Conversation

@GMatuleviciute

Copy link
Copy Markdown
Contributor

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

Implementation

Added internal visibility for Digital Vouchers to access E-Documents.

Important

This Pull request related with microsoft/ALAppExtensions#29793

AndriusAndrulevicius and others added 6 commits February 24, 2026 11:23
Introduces new integration events to enable external handlers after creating an exported e-document and after updating a posted purchase e-document.
Enhances extensibility for partners and custom solutions by allowing additional logic to be executed at these key points.
Adjusts the timing of the post-creation event to ensure it is called after the document and service statuses have been updated.
This change helps external handlers to access the latest status information, improving reliability for event subscribers.
@GMatuleviciute GMatuleviciute requested review from a team June 19, 2026 08:29
@GMatuleviciute GMatuleviciute requested review from a team as code owners June 19, 2026 08:29
@github-actions github-actions Bot added AL: Apps (W1) Add-on apps for W1 From Fork Pull request is coming from a fork labels Jun 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

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.

@ventselartur

Copy link
Copy Markdown
Contributor

@GMatuleviciute, could you please elaborate a bit on this change? Why this change is required and how it is related to the PR? microsoft/ALAppExtensions#29793

I think we have been talking that E-Document Core should not take a dependency on Digital vouchers. It is other way around. Digital voucher can have a dependency from E-Document Core if needed.

@GMatuleviciute

GMatuleviciute commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

@GMatuleviciute, could you please elaborate a bit on this change? Why this change is required and how it is related to the PR? microsoft/ALAppExtensions#29793

I think we have been talking that E-Document Core should not take a dependency on Digital vouchers. It is other way around. Digital voucher can have a dependency from E-Document Core if needed.

@ventselartur Sure, I have added internal visibility, but not a dependency, to the E-Documents. Even though we created events to subscribe to E-Documents, we still need internal visibility to access them. This is because the codeunits that need to be subscribed to in the E-Doc solution are internal.
Code in Digital Voutchers:
image
E-Doc codeunit:
image

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 From Fork Pull request is coming from a fork needs-approval Workflow runs require maintainer approval to start

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants