Repository navigation
[E-Documents Core] - Internal visibility for Digital Vouchers - #8677
Grasiele Matuleviciute (GMatuleviciute) wants to merge 6 commits into
Conversation
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.
github-actions
Bot
added
AL: Apps (W1)
|
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. |
github-actions
Bot
added
the
needs-approval
|
Grasiele Matuleviciute (@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. |
Jesper Schulz-Wedde (JesperSchulz)
added
the
Team: Finance
Anders (AndersLarsenMicrosoft)
added
Team: Integrations
|
Closing as we wont be giving internal visibility to. |
|
Make the events public. Internal visible to is incorrect deisgn. |
Magnus Hartvig Grønbech (@Groenbech96) The property internalsVisibleTo is needed not only because of the events but also because we're using the procedure |
|
Magnus Hartvig Grønbech (@Groenbech96) rebuild solution to avoid internal visibility and migrated Digital Vouchers changes from ALAppExtensions repos. You can find changes here: #9208 As this pull request is now outdated, I'm closing it. |
This pull request does not have a related issue as it's part of the delivery for development agreed directly with Aleksandar Totovic (@altotovi) Magnus Hartvig Grønbech (@Groenbech96)
Implementation
Added internal visibility for Digital Vouchers to access E-Documents.
Important
This Pull request related with microsoft/ALAppExtensions#29793