Skip to content

uha fix susceptibility chart#4885

Merged
one-community merged 4 commits intodevelopmentfrom
fix-susceptibility-chart
Apr 11, 2026
Merged

uha fix susceptibility chart#4885
one-community merged 4 commits intodevelopmentfrom
fix-susceptibility-chart

Conversation

@uhakruthi1
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-02-21 at 7 07 58 PM Screenshot 2026-02-21 at 7 08 07 PM

Related PRS (if any):

This frontend PR is related to the development backend PR.

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 dashboard→mostsusceptibletoolschart

Screenshots or videos of changes:

Screenshot 2026-02-21 at 6 59 01 PM Screenshot 2026-02-21 at 6 59 16 PM

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 22, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 9fe9daa
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69d9b23f25e15d00077cb373
😎 Deploy Preview https://deploy-preview-4885--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 Markdown

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Mar 25, 2026
@Nirali164
Copy link
Copy Markdown

Hello Uha,

I have used your branch and checked the feature locally. It seems to be working for me.
I have tested with various filters.
Screenshot 2026-04-09 at 2 33 21 PM
Screenshot 2026-04-09 at 2 33 06 PM
Screenshot 2026-04-09 at 2 32 47 PM

# Conflicts:
#	src/components/BMDashboard/Tools/SimpleToolChart.jsx
#	yarn.lock
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Uha,

I tested this PR locally and everything looks good overall.

One observation: the date fields currently require manual entry. A date picker component would improve the user experience and reduce the chance of invalid input.

Image Image Image Image

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit ece81b0 into development Apr 11, 2026
10 checks passed
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.

4 participants