Skip to content

Conversation

@daniel-williams-fenwick
Copy link
Contributor

@daniel-williams-fenwick daniel-williams-fenwick commented Sep 25, 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 #4869

Related Base App PR - https://github.com/microsoft/BusinessCentralApps/pull/1363.

Fixes AB#591379

… for the ExcelReports app that was migrated to a new repository.
@github-actions github-actions bot added the From Fork Pull request is coming from a fork label Sep 25, 2025
@daniel-williams-fenwick
Copy link
Contributor Author

@AndreasHans @JesperSchulz I tried to resolve conflicts in the ALAppExtensions repo PR, but it looks like the Excel Reports app has been migrated to this repo.

This PR has been replicated from there.

Andreas, I have also applied the id range updates and id conflict fixes per your latest comment here - microsoft/ALAppExtensions#28526 (comment).

I have closed that PR and issue.

@AndreasHans AndreasHans self-assigned this Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
AndreasHans added a commit that referenced this pull request Sep 26, 2025
@github-actions github-actions bot added the Linked Issue is linked to a Azure Boards work item label Sep 26, 2025
@github-actions github-actions bot added this to the Version 28.0 milestone Sep 26, 2025
@JesperSchulz JesperSchulz enabled auto-merge (squash) September 26, 2025 10:56
@JesperSchulz JesperSchulz merged commit 428e5f3 into microsoft:main Sep 26, 2025
24 of 25 checks passed
@daniel-williams-fenwick daniel-williams-fenwick deleted the task/deprecate-more-finance-rdlc-reports branch September 28, 2025 22:19
miljance pushed a commit to miljance/BCApps that referenced this pull request Oct 22, 2025
…#4870)

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Replace references to deprecated RDLC finance reports with new Excel
reports.

![image](https://github.com/user-attachments/assets/c5b52942-3e81-4e08-8453-ec83b88f0c94)
Some captions remain as (Preview) rather than (Excel) as clean tags
already existed.

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes microsoft#4869

Related Base App PR -
microsoft/BusinessCentralApps#1363.

Fixes
[AB#591379](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/591379)

Co-authored-by: Dan Williams <dan.williams@fenwick.com.au>
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 From Fork Pull request is coming from a fork Linked Issue is linked to a Azure Boards work item

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BC Idea]: Replace references to deprecated RDLC finance reports with new Excel reports

3 participants