Skip to content

[E-Document] Introduce eligibility interface to allow control of when to export#5489

Merged
Groenbech96 merged 3 commits intomainfrom
private/magnushar/exporteligibility
Nov 13, 2025
Merged

[E-Document] Introduce eligibility interface to allow control of when to export#5489
Groenbech96 merged 3 commits intomainfrom
private/magnushar/exporteligibility

Conversation

@Groenbech96
Copy link
Copy Markdown
Contributor

@Groenbech96 Groenbech96 commented Nov 12, 2025

Summary

Moving code from GetTypeFromPostedSourceDocument to single function GetTypeFromSourceDocument, that now converts both posted and non posted types.

IsDocumentTypeSupported is now called IsDocumentSupported, and checks first the type using the GetTypeFromSourceDocument, and then also checks the ShouldExport impl. This works for both check and create methods.
Only check when you want to export.

Add default enum impl on service as a field, that can be set for a given service record, allowing control for partners to determine if documents should be exported or not.

Fixes AB#612844

@Groenbech96 Groenbech96 requested review from a team as code owners November 12, 2025 11:49
@github-actions github-actions bot added the AL: Apps (W1) Add-on apps for W1 label Nov 12, 2025
nikolakukrika
nikolakukrika previously approved these changes Nov 12, 2025
aholstrup1
aholstrup1 previously approved these changes Nov 12, 2025
@Groenbech96 Groenbech96 dismissed stale reviews from aholstrup1 and nikolakukrika via 05c5e0b November 12, 2025 15:42
@github-actions github-actions bot added this to the Version 28.0 milestone Nov 12, 2025
@Groenbech96 Groenbech96 enabled auto-merge (squash) November 13, 2025 11:54
Copy link
Copy Markdown
Contributor

@gggdttt gggdttt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Really like your new design

@Groenbech96 Groenbech96 merged commit 790736c into main Nov 13, 2025
52 of 53 checks passed
@Groenbech96 Groenbech96 deleted the private/magnushar/exporteligibility branch November 13, 2025 12:36
@kasperdj
Copy link
Copy Markdown
Contributor

kasperdj commented Nov 13, 2025

Please note that Jesper has previously promised this to be released in BC27.1 on microsoft/ALAppExtensions#28877. I can see that the issue is now labeled with 28.0! I expect this to be part of 27.2. Thx

@kasperdj
Copy link
Copy Markdown
Contributor

kasperdj commented Dec 1, 2025

@Groenbech96 and I am still missing feedback on my comment from 3 weeks ago regarding the release: #5489 (comment)

@kasperdj
Copy link
Copy Markdown
Contributor

@Groenbech96 and @JesperSchulz - I am still missing your feedback on the release schedule. Please remember that @JesperSchulz originally promised this feature to be released with BC27.1

@JesperSchulz
Copy link
Copy Markdown
Contributor

@kasperdj, I promised to backport microsoft/ALAppExtensions#28877 - and I did! Those changes have been released with 27.1.
I don't think I've been in over this PR, neither have I had a backport of this code on my radar. But I can try to backport it now for release with 27.3. The deadline is today.

JesperSchulz pushed a commit that referenced this pull request Jan 8, 2026
… to export (#5489)

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->

Moving code from GetTypeFromPostedSourceDocument to single function
GetTypeFromSourceDocument, that now converts both posted and non posted
types.

IsDocumentTypeSupported is now called IsDocumentSupported, and checks
first the type using the GetTypeFromSourceDocument, and then also checks
the ShouldExport impl. This works for both check and create methods.
Only check when you want to export.

Add default enum impl on service as a field, that can be set for a given
service record, allowing control for partners to determine if documents
should be exported or not.

Fixes
[AB#612844](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/612844)

---------

Co-authored-by: Magnus Hartvig Grønbech <magnushar@microsoft.com>
@kasperdj
Copy link
Copy Markdown
Contributor

kasperdj commented Jan 8, 2026

@JesperSchulz please do that. Thx!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants