Skip to content

Neeraj Add Get List Feature, No-Show List Modal And Tests For Drop-Off Tracking#4564

Merged
one-community merged 2 commits intodevelopmentfrom
Neeraj_NoShow_GetList_Feature
Apr 5, 2026
Merged

Neeraj Add Get List Feature, No-Show List Modal And Tests For Drop-Off Tracking#4564
one-community merged 2 commits intodevelopmentfrom
Neeraj_NoShow_GetList_Feature

Conversation

@Neeraj-Kondaveeti
Copy link
Copy Markdown
Contributor

@Neeraj-Kondaveeti Neeraj-Kondaveeti commented Dec 16, 2025

Description

image image image image

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

  • Added a Get List action to each row in the Drop-off and no-show rate tracking table.
  • Implemented the no-show list modal that opens when the Get List button is clicked.
  • Built the no-show list modal using mock data, including selectable users.
  • Added the Send Email CTA as a visual placeholder only (no functional implementation in this PR).
  • Added frontend unit tests for the DropOffTracking component covering rendering, headers, and interactions.

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. Navigate to: http://localhost:5173/communityportal/reports/participation
  6. Verify the Drop-off and no-show rate tracking section renders correctly.
  7. Confirm the Get List (👥) button appears for each event row.
  8. Verify the no-show list modal opens with mock user data and selectable checkboxes.

Screenshots or videos of changes:

image image image

Note:

  • This PR intentionally combines two closely related tasks (UI + interaction logic) for the same Get List feature.
  • The Send Email CTA is not implemented in this PR and is included for UI completeness only.
  • All changes are frontend-only and rely on existing mock data.
  • Unit tests were added only for the modified frontend components, in line with current coverage requirements.

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 16, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 05ff331
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69c0d0705748070008e3ad98
😎 Deploy Preview https://deploy-preview-4564--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.

@Neeraj-Kondaveeti Neeraj-Kondaveeti added the do not review Do not review or look at code without full context label Dec 16, 2025
@sonarqubecloud
Copy link
Copy Markdown

@Neeraj-Kondaveeti Neeraj-Kondaveeti added High Priority and removed do not review Do not review or look at code without full context labels Dec 16, 2025
@beblicarl beblicarl self-requested a review February 5, 2026 10:57
Copy link
Copy Markdown
Contributor

@beblicarl beblicarl left a comment

Choose a reason for hiding this comment

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

This feature works perfectly on light mode; however, on dark mode, the writing on the buttons isn't visible

Image Image

Copy link
Copy Markdown

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Neeraj,

Well done with this implementation. Dark mode needs fixing.

Image

@one-community one-community added High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible and removed High Priority labels Mar 21, 2026
Select all
</label>

{mockEvents.slice(0, 8).map(user => (
Copy link
Copy Markdown

@naznin07 naznin07 Mar 22, 2026

Choose a reason for hiding this comment

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

mockEvents data modeling.

PR # 4564

@Neeraj-Kondaveeti Neeraj-Kondaveeti force-pushed the Neeraj_NoShow_GetList_Feature branch from c8f8ff2 to 05ff331 Compare March 23, 2026 05:32
@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 Neeraj,

I checked this PR locally, and everything is working fine as expected according to the requirements.

Image Image Image

Copy link
Copy Markdown

@Vishnu10-hub Vishnu10-hub left a comment

Choose a reason for hiding this comment

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

Hi,

Tested locally on Neeraj_NoShow_GetList_Feature branch.
Participation page loads correctly, Get List (👥) button is visible, and modal opens with selectable user data.
Feature is working as expected.

Image Image Image

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 5893199 into development Apr 5, 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.

7 participants