Skip to content

Added financial report definitions for US localization demo data #28930

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Added financial report definitions for US localization demo data #28930

wants to merge 13 commits into from

Conversation

@tuan-nguyen-fenwick
Copy link

@tuan-nguyen-fenwick tuan-nguyen-fenwick commented Jul 24, 2025 •
edited by github-actions bot
Loading

Summary

New report definitions

  • BS: Balance Sheet
  • BS AUDIT: Balance Sheet Lead Audit Schedule
  • IS: Income Statement
  • IS AUDIT: Income Statement Audit Lead Schedule

Work Item(s)

Fixes #28912
Fixes AB#575838

Tuan Nguyen added 10 commits July 1, 2025 11:36
Added new row definitions
d218f05
Added schedule lines for Balance Sheet row definition (US only)
7c3ddd8
Added schedule lines for BS Audit & Income Statement row definitions …
8f1da76 
…(US only)
Added Indentation parameter to set account schedule line's indentation
100f460
Added financial report definitions for US demo data
738ff8d
Revert "Added new row definitions"
f90aa21 
This reverts commit d218f05.
Added row definitions for US demo data
5b785bd
Fixed typo in the row descriptions
8d35add
Fixed Liabilities typo
9c91155
Merged PR 36082: Added row & financial report definitions for US loca…
407aadb 
…lization demo data

[New column-row-report definitions.xlsx](https://dev.azure.com/fenwick/bf100862-1f7e-4467-bf86-526d4c7ac7eb/_apis/git/repositories/bc6a2808-2da7-4039-8a77-6283538e4d5d/pullRequests/36082/attachments/New%20column-row-report%20definitions.xlsx)

Related work items: microsoft#26680
@tuan-nguyen-fenwick tuan-nguyen-fenwick requested a review from a team as a code owner July 24, 2025 04:49
@JesperSchulz JesperSchulz added the Finance GitHub request for Finance area label Aug 5, 2025
@github-actions github-actions bot added the linked Issue is linked to a Azure Boards work item label Aug 6, 2025
@AleksanderGladkov AleksanderGladkov added Integration GitHub request for Integration area and removed Finance GitHub request for Finance area labels Aug 7, 2025
@tuan-nguyen-fenwick
Copy link
Author

@MonicaPAhuja Could you review this PR please? thank you

@JesperSchulz
Copy link
Contributor

@MonicaPAhuja Could you review this PR please? thank you

Please disregard the random assignment of code reviewers. Not sure why GitHub does that.

@WaelAbuSeada, we need your review of this one.

@JesperSchulz JesperSchulz reopened this Aug 20, 2025
@tuan-nguyen-fenwick
Copy link
Author

@WaelAbuSeada I've fixed up the missing functions.

image

@tuan-nguyen-fenwick
Copy link
Author

@WaelAbuSeada I've updated and confirmed the fix:

image

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

Reviewers

@MonicaPAhuja MonicaPAhuja Awaiting requested review from MonicaPAhuja MonicaPAhuja is a code owner automatically assigned from microsoft/dynamics-365-business-central

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Integration GitHub request for Integration area linked Issue is linked to a Azure Boards work item

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add row and financial report definitions for US localization in the Demo Data tool

3 participants

@tuan-nguyen-fenwick @JesperSchulz @AleksanderGladkov