Skip to content

Rishitha phase3 visualizations for no show rate reports#3156

Merged
one-community merged 13 commits intodevelopmentfrom
Rishitha_phase3_Visualizations_for_No-Show_Rate_Reports
Jun 7, 2025
Merged

Rishitha phase3 visualizations for no show rate reports#3156
one-community merged 13 commits intodevelopmentfrom
Rishitha_phase3_Visualizations_for_No-Show_Rate_Reports

Conversation

@RishithaMamidala
Copy link
Copy Markdown
Contributor

@RishithaMamidala RishithaMamidala commented Feb 12, 2025

Description

(PRIORITY MEDIUM) Yash :Phase 3 - Develop Front-End Visualizations for No-Show Rate Reports (WIP Rishitha Mamidala)
URL : /communityportal/reports/participation
Details:
Design demographic breakdown charts for no-show rates.
Create comparison visuals to identify trends across event types and locations.

image

Related PRS (if any):

This frontend PR is related to the https://github.com/OneCommunityGlobal/HGNRest/pull/1227 backend PR.
To test this frontend PR you need to checkout the https://github.com/OneCommunityGlobal/HGNRest/pull/1227 backendPR.

Main changes explained:

  • Create src/actions/communityPortal/NoShowVizActions.js for no show visual actions
  • Create src/components/CommunityPortal/Attendence/NoshowViz.css for styling
  • Create src/components/CommunityPortal/Attendence/NoshowViz.jsx for no show visual rendering

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 http://localhost:3000/communityportal/reports/participation
  6. verify all the charts are rendered correctly

Note:

Be sure to checkout backend PR 1227
Run " npm run build " in backend PR [Orelse the PR will not work]

The database is not yet setup for this, So, charts are rendered based on mock data in the backend file mockattendencedata.jsx

Screenshots or videos of changes:

video1692588005.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 12, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit f14af80
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6840efd805bcc7000882276f
😎 Deploy Preview https://deploy-preview-3156--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.

@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 Feb 13, 2025
Comment thread src/components/CommunityPortal/Attendence/NoshowViz.jsx
AurHubertMax
AurHubertMax previously approved these changes Feb 15, 2025
Copy link
Copy Markdown
Contributor

@AurHubertMax AurHubertMax left a comment

Choose a reason for hiding this comment

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

Followed instructions, graphs function as expected, animations are smooth as well. However, dark mode is not reflected in the page.
Summary of conditions tested:

  • Checked graph UI, animation, and hovers in light mode
  • Checked graph UI, animation, and hovers in dark mode
  • Checked animations and displays in different screen sizes
2-Click.Screen.Recorder-20250214-210424.891.mp4
2-Click.Screen.Recorder-20250214-210828.822.mp4

Akshay-Jayaram
Akshay-Jayaram previously approved these changes Feb 15, 2025
Copy link
Copy Markdown
Contributor

@Akshay-Jayaram Akshay-Jayaram left a comment

Choose a reason for hiding this comment

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

This PR works correctly and is responsive. Graphs function as expected, animations are smooth, and the UI adapts well to different screen sizes.

Screen.Recording.2025-02-15.at.12.52.17.PM.mov

SunilKotte
SunilKotte previously approved these changes Feb 16, 2025
Copy link
Copy Markdown

@SunilKotte SunilKotte left a comment

Choose a reason for hiding this comment

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

PR #3156 is working as expected
check into current branch
do npm install and ... to run this PR locally
Clear site data/cache
log as admin user
Go to http://localhost:3000/communityportal/reports/participation
verify all the charts are rendered correctly

PR#3156+#1227

Manoj99Q
Manoj99Q previously approved these changes Feb 18, 2025
Copy link
Copy Markdown
Contributor

@Manoj99Q Manoj99Q left a comment

Choose a reason for hiding this comment

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

checked out the related backend PR #1227 along with
PR #3156 Logged in as admin user
Went to http://localhost:3000/communityportal/reports/participation
The graphs are rendering correctly and are responsive.
https://github.com/user-attachments/assets/a36ff722-cb4c-4abf-8d85-b4f7e099e418

jeyanthi-sm
jeyanthi-sm previously approved these changes Feb 19, 2025
Copy link
Copy Markdown
Contributor

@jeyanthi-sm jeyanthi-sm left a comment

Choose a reason for hiding this comment

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

I have tested this PR for the following details:
Designed demographic breakdown charts for no-show rates.
Created comparison visuals to identify trends across event types and locations.
The graphs are rendering nicely. However, my only concern is that the title do not match the images in the description
PR 3156-1227

Copy link
Copy Markdown
Contributor

@jaissica jaissica left a comment

Choose a reason for hiding this comment

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

The PR functions correctly and is fully responsive. Graphs perform as expected, animations run smoothly, and the UI adjusts well across various screen sizes. Just a minor tweak is needed — the content and line overlap in the pie chart, as shown in the screenshot.
Screenshot 2025-02-19 at 12 24 47 PM

Screen.Recording.2025-02-19.at.12.22.06.PM.mov

sundarmachani
sundarmachani previously approved these changes Feb 19, 2025
Copy link
Copy Markdown
Contributor

@sundarmachani sundarmachani left a comment

Choose a reason for hiding this comment

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

Followed the tesing procedure as prescribed and Tested this PR along side with Backend PR #1227 the visualization is working as expected with the features.

Screenshot 2025-02-19 at 5 42 36 PM

@koushi1906 koushi1906 assigned koushi1906 and unassigned koushi1906 Feb 20, 2025
@koushi1906
Copy link
Copy Markdown

PR #3156 is rendering the visualizations responsively. Every thing looks perfect.
The only change requested is to manage the overlap between content and the pie chart.
Screenshot 2025-02-19 at 10 54 44 PM

Copy link
Copy Markdown
Contributor

@dipti95 dipti95 left a comment

Choose a reason for hiding this comment

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

I tested this PR along with backend PR #1227 and reviewed the demographic breakdown charts for no-show rates. I also checked the comparison visuals to identify trends across event types and locations. The graphs are rendering well; however, my only concern is that the titles do not match the images in the description.

prNumber_frontend_.3156.mp4

@saishekhar10
Copy link
Copy Markdown
Contributor

I've tested this PR along with the backend PR #1227. The graphs are not rendering at all, I'm not sure why this is happening.
I've logged in as an admin user, used this PR on the frontend and #1227 on the backend.
I'm getting 404 ( not found ) error on the console
https://github.com/user-attachments/assets/59ff6f70-d75a-4836-bbe9-90cc6c8b6d58

@saishekhar10 saishekhar10 self-requested a review February 21, 2025 22:54
saishekhar10
saishekhar10 previously approved these changes Feb 21, 2025
Copy link
Copy Markdown
Contributor

@saishekhar10 saishekhar10 left a comment

Choose a reason for hiding this comment

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

I tested this PR along with the backend PR #1227 and everything is working as expected, graphs are being rendered and UI animations seem to working correctly.

Screen.Recording.2025-02-21.at.5.57.14.PM.mov

@kkkaiaa
Copy link
Copy Markdown
Contributor

kkkaiaa commented Mar 1, 2025

Reviewed the frontend PR 3156. The graphs all work as expected, and the code is well-structured
截屏2025-03-01 下午3 24 17
截屏2025-03-01 下午3 26 09

SSP24SCM16G
SSP24SCM16G previously approved these changes Mar 1, 2025
Copy link
Copy Markdown

@SSP24SCM16G SSP24SCM16G left a comment

Choose a reason for hiding this comment

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

The backend PR #1227 and frontend PR #3156 have been reviewed and tested together. Logged in as an admin user and navigated to http://localhost:3000/communityportal/reports/participation to verify functionality. The graphs render correctly, are fully responsive, and animations run smoothly without any lag. The UI adapts well across different screen sizes with no noticeable glitches or performance issues. Overall, the implementation works as expected, and no issues were found. The PR is approved and ready to merge.

https://drive.google.com/file/d/14HlgSv4D9xYdPuv8-49Bszc0fJEiQOXp/view?usp=sharing

Copy link
Copy Markdown
Contributor

@barnaboss305 barnaboss305 left a comment

Choose a reason for hiding this comment

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

The graphs look good and the page functions as expected, but the visuals of the page do not change in dark mode.
Screenshot 2025-03-01 at 4 47 36 PM

@Guirong-Wu
Copy link
Copy Markdown

PR reviewed. The charts are rendered correctly, are fully responsive, and the animations run smoothly which is excellent. But it does not change in dark mode. Overall this implementation works functionally.
3156

vasavi-139
vasavi-139 previously approved these changes Mar 7, 2025
Copy link
Copy Markdown
Contributor

@vasavi-139 vasavi-139 left a comment

Choose a reason for hiding this comment

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

Functionality works as expected.
Screenshot 2025-03-07 140014
Screenshot 2025-03-07 140025
All the charts are rendered correctly

rishwa344
rishwa344 previously approved these changes Mar 8, 2025
Copy link
Copy Markdown
Contributor

@rishwa344 rishwa344 left a comment

Choose a reason for hiding this comment

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

The UI is working as expected and smoothly
Screenshot 2025-03-07 at 11 35 06 PM
Screenshot 2025-03-07 at 11 40 42 PM

@HarideepValiveru
Copy link
Copy Markdown

Reviewed PR, it introduces well-structured constants, reducers and API end points. The code is organised, scalable and maintains consistency. It works well as expected

@RishithaMamidala RishithaMamidala dismissed stale reviews from rishwa344 and vasavi-139 via 6ed65fc March 18, 2025 17:53
@one-community
Copy link
Copy Markdown
Member

I've tested this PR along with the backend PR #1227. The graphs are not rendering at all, I'm not sure why this is happening. I've logged in as an admin user, used this PR on the frontend and #1227 on the backend. I'm getting 404 ( not found ) error on the console https://github.com/user-attachments/assets/59ff6f70-d75a-4836-bbe9-90cc6c8b6d58

This should be a requested change, not just a comment.

@Yash-OCG
Copy link
Copy Markdown

Yash-OCG commented May 4, 2025

@RishithaMamidala
Plz address the requested changes and try resolving it

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit a08cd33 into development Jun 7, 2025
6 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.