Skip to content

Conversation

@daniel-williams-fenwick
Copy link
Contributor

@daniel-williams-fenwick daniel-williams-fenwick commented Apr 15, 2025

Summary

Replace references to deprecated RDLC finance reports with new Excel reports.
image
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

@daniel-williams-fenwick daniel-williams-fenwick requested a review from a team as a code owner April 15, 2025 05:27
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Apr 24, 2025
@microsoft microsoft deleted a comment from github-actions bot Jun 5, 2025
@JesperSchulz JesperSchulz added linked Issue is linked to a Azure Boards work item processing-PR The PR is currently being reviewed labels Jul 7, 2025
@JesperSchulz JesperSchulz self-assigned this Jul 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 7, 2025

Processing this PR. The branch is now locked 🔒 Please don't push updates unless otherwise agreed.

@JesperSchulz
Copy link
Contributor

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.

@daniel-williams-fenwick
Copy link
Contributor Author

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.

daniel-williams-fenwick and others added 3 commits August 7, 2025 09:15
…ist.PageExt.al


Fixed "Customer List" id conflict.

Co-authored-by: Jesper Schulz-Wedde <JesperSchulz@users.noreply.github.com>
@daniel-williams-fenwick
Copy link
Contributor Author

Also looks like some local builds don't compile with this:

App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountingManagerRC.PageExt.al(36,18): error AL0271: The action 'Action115' is not found in the target 'Accounting Manager Role Center'
App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRBookkeeperRC.PageExt.al(36,18): error AL0271: The action 'Action49' is not found in the target 'Bookkeeper Role Center'

@JesperSchulz I've changed where these addafters point to at Yusheng's suggestion and am praying they exist in all locales.

Dan Williams added 2 commits August 7, 2025 11:25
@JesperSchulz JesperSchulz removed their assignment Sep 2, 2025
@AndreasHans AndreasHans self-assigned this Sep 12, 2025
AndreasHans added a commit that referenced this pull request Sep 12, 2025
AndreasHans added a commit that referenced this pull request Sep 12, 2025
AndreasHans added a commit that referenced this pull request Sep 12, 2025
AndreasHans added a commit that referenced this pull request Sep 12, 2025
@AndreasHans
Copy link
Contributor

In app.json please change the range from [4400..4420] to [4400..4440]

@AndreasHans
Copy link
Contributor

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.

App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountantRoleCenter.PageExt.al(84,18): error AL0271: The action '"G/L Reports"' is not found in the target 'Accountant Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountantRoleCenter.PageExt.al(95,18): error AL0271: The action '"Bu&dget"' is not found in the target 'Accountant Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountantRoleCenter.PageExt.al(106,18): error AL0271: The action '"Customers and Vendors"' is not found in the target 'Accountant Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountingManagerRC.PageExt.al(25,18): error AL0271: The action '"Bu&dget"' is not found in the target 'Accounting Manager Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRBookkeeperRC.PageExt.al(14,18): error AL0271: The action '"A&ccount Schedule"' is not found in the target 'Bookkeeper Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRPurchasingManagerRC.PageExt.al(13,18): error AL0271: The action '"Vendor - Detail Trial Balance"' is not found in the target 'Purchasing Manager Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRGLAccountCard.PageExt.al(13,18): error AL0271: The action '"Detail Trial Balance"' is not found in the target 'G/L Account Card' App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRChartOfAccounts.PageExt.al(13,18): error AL0271: The action '"Detail Trial Balance"' is not found in the target 'Chart of Accounts' App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRChartOfAccounts.PageExt.al(24,18): error AL0271: The action '"Detail Trial Balance_Promoted"' is not found in the target 'Chart of Accounts' .

Additionally:
App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRChartOfAccounts.PageExt.al(26,23): error AL0725: The action type 'ActionRef' is not allowed in area 'None'
Here the action ref you want to add it after does not exist in some cases. Use use Category_Report in this case and addlast.

Dan Williams added 2 commits September 18, 2025 09:25
…st in certain localisations with AddLast(reporting). Added AllowInCustomizations = Never; where required.
@daniel-williams-fenwick
Copy link
Contributor Author

In app.json please change the range from [4400..4420] to [4400..4440]

Updated

@daniel-williams-fenwick
Copy link
Contributor Author

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.

App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountantRoleCenter.PageExt.al(84,18): error AL0271: The action '"G/L Reports"' is not found in the target 'Accountant Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountantRoleCenter.PageExt.al(95,18): error AL0271: The action '"Bu&dget"' is not found in the target 'Accountant Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountantRoleCenter.PageExt.al(106,18): error AL0271: The action '"Customers and Vendors"' is not found in the target 'Accountant Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRAccountingManagerRC.PageExt.al(25,18): error AL0271: The action '"Bu&dget"' is not found in the target 'Accounting Manager Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRBookkeeperRC.PageExt.al(14,18): error AL0271: The action '"A&ccount Schedule"' is not found in the target 'Bookkeeper Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\RoleCenters\EXRPurchasingManagerRC.PageExt.al(13,18): error AL0271: The action '"Vendor - Detail Trial Balance"' is not found in the target 'Purchasing Manager Role Center' App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRGLAccountCard.PageExt.al(13,18): error AL0271: The action '"Detail Trial Balance"' is not found in the target 'G/L Account Card' App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRChartOfAccounts.PageExt.al(13,18): error AL0271: The action '"Detail Trial Balance"' is not found in the target 'Chart of Accounts' App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRChartOfAccounts.PageExt.al(24,18): error AL0271: The action '"Detail Trial Balance_Promoted"' is not found in the target 'Chart of Accounts' .

Additionally: App\Apps\W1\ExcelReports\TempSource\app\src\Financials\PageExtensions\EXRChartOfAccounts.PageExt.al(26,23): error AL0725: The action type 'ActionRef' is not allowed in area 'None' Here the action ref you want to add it after does not exist in some cases. Use use Category_Report in this case and addlast.

@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.

@AndreasHans
Copy link
Contributor

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 added a commit that referenced this pull request Sep 19, 2025
@daniel-williams-fenwick
Copy link
Contributor Author

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.

@AndreasHans
Copy link
Contributor

Found some more ids that needs update:
4411 => 4440
4413 => 4441
4424 => 4442

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

@AndreasHans
Copy link
Contributor

Please resolve branch conflicts

@daniel-williams-fenwick
Copy link
Contributor Author

ExcelReports app migrated. PR replicated in that repo here - microsoft/BCApps#4870

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Finance GitHub request for Finance area linked Issue is linked to a Azure Boards work item processing-PR The PR is currently being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace references to deprecated RDLC finance reports with new Excel reports

4 participants