-
Notifications
You must be signed in to change notification settings - Fork 687
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.
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
base: main
Are you sure you want to change the base?
Added financial report definitions for US localization demo data #28930
Conversation
This reverts commit d218f05.
…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
|
@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. |
|
@WaelAbuSeada I've fixed up the missing functions.
|
|
@WaelAbuSeada I've updated and confirmed the fix:
|
|
@attilatoury I think it is the same for this PR. The PR is open but the changes are already in main. Should we close it too? |


Summary
New report definitions
Work Item(s)
Fixes #28912
Fixes AB#575838