Skip to content

feat(financial): introduce flattened and full detail financial endpoints - #25

Merged
jSylvestre merged 2 commits into
mainfrom
JCS/SimplifiedDetails20260901
Sep 2, 2026
Merged

jSylvestre merged 2 commits into
mainfrom
JCS/SimplifiedDetails20260901

Conversation

@jSylvestre

Copy link
Copy Markdown
Member

The existing /api/v1/financial/details endpoints now provide a flattened view of Aggie Enterprise data, optimized for simpler consumption, such as in Kuali Build. This involves mapping complex nested structures to top-level properties and concatenating errors and warnings into single string fields.

New /api/v1/financial/full-details endpoints are introduced to return the complete, original Aggie Enterprise financial response graph, preserving all structural details.

The existing `/api/v1/financial/details` endpoints now provide a flattened view of Aggie Enterprise data, optimized for simpler consumption, such as in Kuali Build. This involves mapping complex nested structures to top-level properties and concatenating errors and warnings into single string fields.

New `/api/v1/financial/full-details` endpoints are introduced to return the complete, original Aggie Enterprise financial response graph, preserving all structural details.
Updates the FinancialDetails model to retrieve 'AwardInfo' from the Aggie Enterprise segment details rather than PPM details. This aligns 'AwardInfo' with other segment-based financial fields, ensuring consistent data sourcing.

A new private helper method, 'FindSegmentName', is introduced to encapsulate the logic for looking up segment names by entity, reducing code duplication for fields like 'GlFinancialDepartmentName' and the updated 'AwardInfo'.
@jSylvestre
jSylvestre merged commit 79d91a8 into main Sep 2, 2026
1 check passed
@jSylvestre
jSylvestre deleted the JCS/SimplifiedDetails20260901 branch September 2, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants