Skip to content

Sireesha phase3 participation report fixes#5304

Open
kunchalasireesha wants to merge 9 commits into
developmentfrom
sireesha_phase3_participation_report_fixes
Open

Sireesha phase3 participation report fixes#5304
kunchalasireesha wants to merge 9 commits into
developmentfrom
sireesha_phase3_participation_report_fixes

Conversation

@kunchalasireesha

@kunchalasireesha kunchalasireesha commented May 29, 2026

Copy link
Copy Markdown
Contributor

Description

image This PR addresses issues in Phase 3 – Develop Visualization for HGN Management System. After the previous PR (https://github.com//pull/3052) was merged, the /communityportal/reports/participation page was only showing: The month/year calendar view, and The registered events list only The event registration form/page was not accessible, and several UI elements (charts, insights, navigation, CSS layout) were missing or incomplete. Additionally, CSS modules were not applied, causing inconsistent styling across light/dark mode. This PR completes all missing functionality and resolves the accessibility and routing issues.

Related PRS (if any):

This frontend PR is related to the #3052, #4401 backend PR.
There are no backend PR dependencies

Main changes explained:

  1. Restored Navigation + Routing Fixes

Fixed the routing so /communityportal/reports/participation loads the correct Event Participation landing page.
Enabled navigation to:
Demographics Insights
Personalization Insights
Event Insights (placeholder)
Create New Registration (temporary mock)

  1. Calendar View Completion

Restored the original calendar UI layout.
Implemented event bubbles + event list below the calendar.
Fixed dark mode date visibility.
Ensured weekday labels, tiles, and event dots inherit theme colors.

  1. “+ Create New” Functionality

Button was previously non-functional.
Redirect now added (temporary placeholder page included).

  1. Added Segmentation Views

Demographics segmentation page
Personalization insight page
Updated navigation UI to link all pages.

How to test:

git checkout sireesha_phase3_participation_report_fixes
npm install
Run frontend locally:
npm run start:local
Clear browser cache + local storage
Login as Admin user
Navigate to:
http://localhost:5173/communityportal/reports/participation
Verify the following:
Event Participation page loads with full layout.

Calendar shows days correctly in both light and dark mode.

Event bubbles appear in the correct dates.

Event details display when a date is selected.

Charts render correctly.

No-show insights tabs function.

“+ Create New” redirects.

Navigation buttons open:

Demographics
Personalization
Event Insights (placeholder)
Verify CSS modules:
No global selector conflicts.
Dark mode applies properly across all sections.

Screenshots or videos of changes:

PR.5304.mov

Note:

Include the information the reviewers need to know.

@netlify

netlify Bot commented May 29, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit c97d9d2
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a34ad3ab5bbdf0008905bd8
😎 Deploy Preview https://deploy-preview-5304--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@kunchalasireesha kunchalasireesha marked this pull request as ready for review June 12, 2026 19:56
@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 Jun 12, 2026

@HemanthNidamanuru HemanthNidamanuru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Sireesha,

I tested this PR locally on the sireesha_phase3_participation_report_fixes branch.
The page layout is getting cropped and hover tooltips are not visible in dark mode. Future events also show "Jan" instead of the correct month. Please let me know if that last one is in scope for this PR or should be tracked separately.

PR.5304.mp4

@DeepighaJ DeepighaJ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • Tested the PR locally and given findings below:
  • The layout needs improvement and dark mode tool tips on all graphs are not visible.
  • Calendar needs improvement on both the mode, dates are not displayed properly.
  • Demographics and Personalization links opens and shows insight placeholder.
  • Create new works as expected.
Image Image Image Image Image

@mahathiganimi mahathiganimi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi, Tested this branch locally and found some issues in Dark mode. When hovered on the charts, tooltip is not working. When navigating to Demographics and personalisation pages, it would be nice to have a back button to revert back to the current page. The colour palette when highlighting certain areas in Dark mode makes users difficult to read. In mobile view, the page looks crammed.

Image

Would also suggest to clear merge conflicts before getting this PR approved. Requesting the said changes

Image Image Image Image Image Image Image

@Abhi-R0211 Abhi-R0211 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I tested locally on branch sireesha_phase3_participation_report_fixes, logged in as admin, navigated to http://localhost:5173/communityportal/reports/participation. Here is my analysis:

What's Working Well

  • Full page layout renders correctly in both light and dark mode — event cards, drop-off/no-show rate tracking, no-show rate insights, comparative charts, monthly trend, and participation by location all present
Image Image
  • Calendar view works in both modes — today's date is highlighted, navigation functions, and the event list for the selected date populates
Image
  • Demographics and Personalization buttons route correctly to their sub-pages
  • Card view renders event cards correctly in both modes

Issues

  1. Dark mode chart tooltips — blank white box instead of data
    Hovering over any chart in dark mode renders a blank white rectangle with no tooltip content
Image
  1. Calendar event list — fields concatenated without spaces
    Event detail text runs together without separators — e.g. "Event 93Fitness Bootcamp" and "Jun 18, 2026, 12:00 AMSan Francisco+88 attendees" — affecting readability in both modes
Image
  1. Demographics and Personalization sub-pages — placeholder only, no back button
    Both pages show only a single placeholder message with no actual content and no way to navigate back to the main participation page
Image
  1. Mobile layout cramped at 375px
    Comparative charts render as very narrow single-column cards with labels barely fitting the viewport
Image Image

…odNetworkApp into sireesha_phase3_participation_report_fixes
@sonarqubecloud

Copy link
Copy Markdown

@beblicarl beblicarl self-requested a review June 19, 2026 08:35

@beblicarl beblicarl 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.

This functionality works as intended, and the code is great

https://www.loom.com/share/950e74307e044eeb88c2d8ad4339f725

@HemanthNidamanuru HemanthNidamanuru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Sireesha,

Retested this PR locally on the sireesha_phase3_participation_report_fixes branch. The previously reported issues (layout cropping, hover tooltips in dark mode, and the "Jan" month display) have been resolved.

However, I found a new issue: Card view and List view are not displaying any data.

Image Image Image Image Image Image Image Image Image

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.

7 participants