Skip to content

Roshini Seelamsetty: Financials Tracking: Duplicate “Actual vs Planned Costs” and “Planned Vs Actual costs tracking” charts showing different values - #2468

Open
roshini1212 wants to merge 1 commit into
developmentfrom
Roshini_Seelamsetty_Financials-Tracking-Duplicate-Actual-vs-Planned-Costs-and-Planned-Vs-Actual-costs-tracking-charts-showing-different-values
Open

roshini1212 wants to merge 1 commit into
developmentfrom
Roshini_Seelamsetty_Financials-Tracking-Duplicate-Actual-vs-Planned-Costs-and-Planned-Vs-Actual-costs-tracking-charts-showing-different-values

Conversation

@roshini1212

@roshini1212 roshini1212 commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-11-05 at 11 22 22

Related PRS (if any):

To test this backend PR you need to checkout the #5779 (OneCommunityGlobal/HighestGoodNetworkApp#5779) frontend PR.

Main changes explained:

  • Update bmActualVsPlannedCostController.js to return actual and planned totals from persisted expenditure records, grouped by category.
  • Remove randomized cost estimates from the endpoint.
  • Add controller tests for expenditure-backed totals, invalid project IDs, projects with no expenditure records, and database errors.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to /bmdashboard/totalconstructionsummary
  6. Under Financials Tracking, verify the duplicate chart pair is gone and that Building 3 / Overall totals match the recorded expenditures.

Screenshots or videos of changes:

After
Screenshot 2026-11-05 at 11 27 06

Before

Before.mov

@sonarqubecloud

sonarqubecloud Bot commented Nov 5, 2026

Copy link
Copy Markdown

This branch has not been deployed

No deployments
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.

1 participant