Skip to content

Phase 2 summary dashboard making the horizontal bar graph of p5 be#1463

Closed
ArtemisNyx3 wants to merge 2 commits into
developmentfrom
Phase-2-Summary-Dashboard--Making-the-Horizontal-Bar-Graph-of-P5-BE
Closed

Phase 2 summary dashboard making the horizontal bar graph of p5 be#1463
ArtemisNyx3 wants to merge 2 commits into
developmentfrom
Phase-2-Summary-Dashboard--Making-the-Horizontal-Bar-Graph-of-P5-BE

Conversation

@ArtemisNyx3

Copy link
Copy Markdown

Description

image
image
image

Related PRS (if any):

To test this backend PR you need to checkout the #3660 frontend PR.

Main changes explained:

  • API to the the neccary data

How to test:

  1. check into current branch
  2. do npm install and npm run start:local to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to dashboard→ Rerports→ total construction summary→…
  6. verify function the graphs
  7. Optionally veriy in postman

API calls for reference

  1. GET /issues/longest-open?projectIds=proj1,proj2&startDate=xxx&endDate=xxx
  2. GET /issues/most-expensive?projectIds=proj1,proj2&startDate=xxx&endDate=xxx

Screenshots or videos of changes:

Note:

Include the information the reviewers need to know.

@HaochengXu-0509

Copy link
Copy Markdown

Works well with pr 3660. Good work.
6 18 1

@Saicharan1505 Saicharan1505 added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Aug 4, 2025

@ShravyaKudlu ShravyaKudlu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good works with #3660
Screenshot from 2025-08-05 20-48-07

@one-community

Copy link
Copy Markdown
Member

Replaced with this PR: #2256

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

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants