Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Document Sharing] Add cancelled parameter and use in word template edit after merge #464

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

stkillen
Copy link
Contributor

@stkillen stkillen commented Jan 4, 2024

Summary

For document sharing, add a state parameter that is set when import of file is cancelled.

Use the state parameter to cancel execution in Word Templates EditDocumentAfterMerge if import is cancelled. This will ensure that attachments are not set to unique in Interaction Templates whenever the import is cancelled.

Work Item(s)

Fixes AB#494806

@stkillen stkillen requested a review from a team as a code owner January 4, 2024 09:27
@github-actions github-actions bot added this to the Version 24.0 milestone Jan 4, 2024
IhorHandziuk
IhorHandziuk previously approved these changes Jan 4, 2024
@aholstrup1 aholstrup1 modified the milestones: Version 24.0, Version 25.0 Feb 26, 2024
@@ -609,6 +609,10 @@ codeunit 9988 "Word Template Impl."
TempDocumentSharing.Insert();

DocumentSharing.Share(TempDocumentSharing);

if TempDocumentSharing.State = TempDocumentSharing.State::Cancelled then
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be nice to cover also this one scenario in tests
\src\System Application\Test\Word Templates\src*
I would say codeunit 130443 "Word Templates Test" is good fit for it

@aholstrup1 aholstrup1 modified the milestones: Version 25.0, Version 26.0 Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants