Obsolete objects and add deprecation notices for PTE upload/install in Extension Management - #10046
Obsolete objects and add deprecation notices for PTE upload/install in Extension Management#10046PooyaKharamesh wants to merge 5 commits into
Conversation
Add the SaaS migration notice for the Business Central admin center and mark the Extension Management, PTE upload, and AppSource installation pages as pending obsolete. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Could not find a linked ADO work item. Please link one by using the pattern 'AB#' followed by the relevant work item number. You may use the 'Fixes' keyword to automatically resolve the work item when the pull request is merged. E.g. 'Fixes AB#1234' |
Keep the deprecation notice and pending obsoletion scoped to Extension Management, PTE upload, and the AppSource installation launcher. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| page 2503 "Extension Installation" | ||
| { | ||
| Extensible = false; | ||
| ObsoleteReason = 'Use the Business Central admin center to install and manage extensions.'; |
There was a problem hiding this comment.
Extension Installation remains part of the non-SaaS install flow: ExtensionMarketplace.Codeunit.al unconditionally instantiates and runs Page "Extension Installation" (via ExtensionInstallationPage.RunModal()), and the same codeunit has explicit non-SaaS handling elsewhere (if not EnvironmentInformation.IsSaaS() then OnOverrideUrl(URL)). Marking the page obsolete with ObsoleteReason = 'Use the Business Central admin center to install and manage extensions.' leaves on-prem consumers of this still-active call path without an actionable replacement.
Knowledge:
👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.31.4
| page 2500 "Extension Management" | ||
| { | ||
| Caption = 'Extension Management'; | ||
| ObsoleteReason = 'Use the Business Central admin center to upload and manage per-tenant extensions.'; |
There was a problem hiding this comment.
Extension Management still runs an explicit non-SaaS display path (IsOnPremDisplay := not IsMarketplaceEnabled or not IsSaaS), and the new deprecation notification only fires if IsSaaS. The ObsoleteReason ('Use the Business Central admin center to upload and manage per-tenant extensions.') names a replacement (the admin center) that is not available to on-prem/non-SaaS consumers of this page, leaving them without an actionable migration path per the obsoletion guidance. Either name a replacement that covers on-prem users too, or hold off marking the page obsolete until the non-SaaS path has one.
Knowledge:
👍 useful · ❤️ especially valuable · 👎 wrong - reply with why · AL review agent v1.31.4
Add a descriptive notification action that opens the admin center extension management documentation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use Read more here for the deprecation and admin center documentation action. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Display the same notice in all deployments because the linked deprecation document includes environment-specific guidance. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
2 participants
Summary
Work item
Deliverable 638999