-
Notifications
You must be signed in to change notification settings - Fork 687
Deprecate more finance rdlc reports #28526
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
Deprecate more finance rdlc reports #28526
Conversation
… created Page Extensions.
…sions and Event Handler Codeunit.
…n names to standard with EXR prefix.
|
Processing this PR. The branch is now locked 🔒 Please don't push updates unless otherwise agreed. |
|
Build error: EXRFinRoleCenter.PageExt.al(2,20): error AS0031: The action with name 'EXRVendorTopList' defined in PageExtension 'EXRFinRoleCenter' was found in the previous version, but is missing in the current extension. This will break dependent extensions. |
Thanks @JesperSchulz I've setup AppSourceCop so I can detect these breaking changes, made the fix and checked for any others. |
…ist.PageExt.al Fixed "Customer List" id conflict. Co-authored-by: Jesper Schulz-Wedde <JesperSchulz@users.noreply.github.com>
@JesperSchulz I've changed where these addafters point to at Yusheng's suggestion and am praying they exist in all locales. |
…recations for the Small Business Report Catalog codeunit and pages.
Apps/W1/ExcelReports/app/src/Customer/PageExtensions/EXRCustomerList.PageExt.al
Show resolved
Hide resolved
|
In app.json please change the range from [4400..4420] to [4400..4440] |
|
Some build errors when trying to build and localize. Some of the actions referenced do not exist in all localizations. Instead I prefer that the groups are used instead and use addlast in these cases. Example use reporting group in case of detail trial balance.
Additionally: |
…st in certain localisations with AddLast(reporting). Added AllowInCustomizations = Never; where required.
Updated |
@AndreasHans I've updated the files with build errors so that all of the new actions use addlast(reporting). Let me know if you'd prefer I apply this to all actions across all of the Page Extensions instead. |
|
Error Message: The application object of type 'PageExtension' with the ID '4409' is defined in multiple apps. The apps are: Sales Order Agent by Microsoft 28.0.2147483647.2106; Dynamics BC Excel Reports by Microsoft 28.0.2147483647.2106. Also no settings file is found |
@AndreasHans I've updated the conflicting page extension id, but you told me to set the idranges to 4400..4440 in an earlier comment. Shouldn't that mean those id ranges are reserved for this app? Could you please specify what the settings file should look like? I haven't created one for Microsoft before. |
|
Found some more ids that needs update: and update the range to: 4400...4445 Ignore the comment about settings.json for now. The id range specify which ids can be used by the app. However, these ranges can overlap with other apps |
|
Please resolve branch conflicts |
|
ExcelReports app migrated. PR replicated in that repo here - microsoft/BCApps#4870 |
Summary
Replace references to deprecated RDLC finance reports with new Excel reports.

Some captions remain as (Preview) rather than (Excel) as clean tags already existed.
Work Item(s)
Fixes #28525
Related Base App PR - https://github.com/microsoft/BusinessCentralApps/pull/1363.
Fixes AB#591379