Skip to content

Conversation

@ChiragBellara
Copy link
Contributor

Description

Screenshot 2025-11-28 at 7 05 32 PM

Related PRS (if any):

This change is NOT related to any other PRs.

Main changes explained:

  • Fixed the link between the graphs.
  • Added new state fixes to ensure that the drop-downs of one graph do no affect the values of the other.

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. navigate to /bmdashboard/totalconstructionsummary
  6. verify that when selecting any value in the project dropdown under Longest Open Issues graph in the Issue Tracking section does not change the Projects dropdown under Quantity of Materials Used dropdown in Material Consumption section.

Screenshots or videos of changes:

Here's what the original issue without the fix was.

Project.Dropdown.Link.Issue.mov

Here's how it should work now.

Project.Dropdown.Link.Fixed.mov

Note:

Currently the screen is using Mock Data and hence selection of values may not have any visible impact on the graphs. But please verify that the Project drop-downs in Longest Open Issues graph in the Issue Tracking section and the Projects dropdown under the Quantity of Materials Used graph in Material Consumption section are working independent of each other.

@netlify
Copy link

netlify bot commented Nov 29, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 7064c75
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/692a6570a4df6200076bbefc
😎 Deploy Preview https://deploy-preview-4472--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link

Copy link

@VijayAnirudh VijayAnirudh left a comment

Choose a reason for hiding this comment

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

image Followed the steps, and it doesn't change when we select any project in longest open issues.

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.

3 participants