Skip to content

Som - fix(DatePicker): add "Ending After" date picker to search filters in CPDashboard.jsx#4428

Merged
one-community merged 9 commits intodevelopmentfrom
som-fix/ending-after-figma-mismatch
Apr 6, 2026
Merged

Som - fix(DatePicker): add "Ending After" date picker to search filters in CPDashboard.jsx#4428
one-community merged 9 commits intodevelopmentfrom
som-fix/ending-after-figma-mismatch

Conversation

@somramnani
Copy link
Copy Markdown
Contributor

@somramnani somramnani commented Nov 20, 2025 •
edited
Loading

Description

Screenshot 2025-11-20 at 3 03 57 PM

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

  • CPDashboard.jsx: Added an “Ending After” date picker, including state logic for validating and filtering events based on selected end date.
  • CPDashboard.module.css: Added and organized styles specific to the DatePicker to replace previous inline styling.

How to test:

  1. Checkout this branch.
  2. Run npm install and start the app locally.
  3. Clear site data/cache.
  4. Log in as an admin user.
  5. Navigate to: /communityportal
  6. Verify that the “Ending After” date picker renders correctly, allows selecting a date, and displays an error when a past date is chosen.

Screenshots or videos of changes:

Live Site
Deployed

Figma
Figma

PR Change:
Light Mode:
Screenshot 2026-03-27 at 3 45 05 PM

Dark Mode:
Screenshot 2026-03-27 at 3 44 53 PM

Notes

  • Please verify that the visual design matches the Figma.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 20, 2025 •
edited
Loading

✅ Deploy Preview for highestgoodnetwork-dev ready!

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

@somramnani somramnani changed the title som-fix(CPDashboard): add "Ending After" date picker to search filters Som - fix(CPDashboard): add "Ending After" date picker to search filters Nov 20, 2025
@Prem203
Copy link
Copy Markdown
Contributor

Prem203 commented Nov 23, 2025

Reviewed the PR with an admin account. The Endinf After filter is now visible after the this weekend radio button.

image

@somramnani somramnani added the do not review Do not review or look at code without full context label Nov 24, 2025
@somramnani somramnani force-pushed the som-fix/ending-after-figma-mismatch branch from 1307440 to 4e5e40f Compare December 2, 2025 19:10
@somramnani somramnani changed the title Som - fix(CPDashboard): add "Ending After" date picker to search filters Som - fix(DatePicker): add "Ending After" date picker to search filters in CPDashboard.jsx Dec 2, 2025
@somramnani somramnani added Medium Priority and removed do not review Do not review or look at code without full context labels Dec 3, 2025
@somramnani somramnani added do not review Do not review or look at code without full context and removed Medium Priority labels Dec 11, 2025
@somramnani somramnani force-pushed the som-fix/ending-after-figma-mismatch branch from 79d8922 to 5eff4e0 Compare December 12, 2025 20:10
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 12, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@somramnani somramnani added Medium Priority and removed do not review Do not review or look at code without full context labels Dec 13, 2025
@beblicarl beblicarl self-requested a review January 8, 2026 16:38
beblicarl
beblicarl requested changes Jan 8, 2026
View reviewed changes
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 is functional however it doesn't look great in dark mode

image image

@somramnani somramnani force-pushed the som-fix/ending-after-figma-mismatch branch from 0b1fc8d to 3448807 Compare January 16, 2026 20:02
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 7, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 27, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@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 Medium Priority labels Mar 28, 2026
Anusha-Gali
Anusha-Gali approved these changes Apr 2, 2026
View reviewed changes
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Som,

I have reviewed your PR locally and as mentioned in description, the date picker is visible and displays a error as mentioned when past date is selected.

Image Image Image Image Image

@one-community
Copy link
Copy Markdown
Member

one-community commented Apr 6, 2026

Thank you all, merging!

@one-community one-community merged commit 2cedfb4 into development Apr 6, 2026
10 checks passed
@one-community one-community mentioned this pull request Apr 8, 2026
Merged
@somramnani somramnani mentioned this pull request Apr 10, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@beblicarl beblicarl Awaiting requested review from beblicarl

@ShreyaMP1999 ShreyaMP1999 Awaiting requested review from ShreyaMP1999

@abdel-lall abdel-lall Awaiting requested review from abdel-lall

@SharadhaKasiviswanathan SharadhaKasiviswanathan Awaiting requested review from SharadhaKasiviswanathan

1 more reviewer

@Anusha-Gali Anusha-Gali Anusha-Gali approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@somramnani @Prem203 @one-community @beblicarl @Anusha-Gali