Skip to content

[Extensibility Request] issue 29518: make document logo position extensible - #10540

Draft
Marko Aleksandric (AleksandricMarko) wants to merge 3 commits into
mainfrom
ai-ext-fix/ext_issue-29518
Draft

Marko Aleksandric (AleksandricMarko) wants to merge 3 commits into
mainfrom
ai-ext-fix/ext_issue-29518

Conversation

@AleksandricMarko

@AleksandricMarko Marko Aleksandric (AleksandricMarko) commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Extensions need to support additional company logo placements on sales documents, but the current setup field is a fixed option. This PR converts the field to an extensible, assignment-compatible enum while retaining the existing values and persisted ordinals.

Source issue repository: microsoft/AlAppExtensions; issue number: 29518

Changes Made

  • Logo Position on Documents - added an extensible enum with the existing logo positions and assignment compatibility.
  • Sales & Receivables Setup.Logo Position on Documents - changed the field from Option to the new enum in W1 and all existing layer counterparts.

Fixes AB#616632

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@AleksandricMarko

Copy link
Copy Markdown
Contributor Author

Copilot it is necessary to check all places where field is used and to determine if it is needed to update usage
procedure SetLogoPosition accepts option, it would be the best to create new one which will accept enum (and which will call existing one)

Co-authored-by: AleksandricMarko <133204513+AleksandricMarko@users.noreply.github.com>

Copilot AI commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Copilot it is necessary to check all places where field is used and to determine if it is needed to update usage
procedure SetLogoPosition accepts option, it would be the best to create new one whi...

Added a new SetLogoPosition overload in FormatDocument.Codeunit.al that accepts Enum "Logo Position on Documents" and delegates to the existing Option-based procedure. All other usages of the field pass it directly to SetLogoPosition, which now resolves to the new overload automatically. See commit e.g. the latest push.

@github-actions github-actions Bot added needs-approval Workflow runs require maintainer approval to start and removed needs-approval Workflow runs require maintainer approval to start labels Aug 23, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 7fd29cf0-bf3b-4d3b-98f6-c1a286323ca8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enum-request Team: SCM GitHub request for SCM area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants