Skip to content

Siva - Restrict past date selection in event search calendar#4338

Merged
one-community merged 15 commits intodevelopmentfrom
siva/fix-past-date-search-issue
Apr 12, 2026
Merged

Siva - Restrict past date selection in event search calendar#4338
one-community merged 15 commits intodevelopmentfrom
siva/fix-past-date-search-issue

Conversation

@pavanputti
Copy link
Copy Markdown
Contributor

@pavanputti pavanputti commented Nov 6, 2025

Description

Screenshot 2025-11-07 at 1 30 47 AM

Related PRS (if any):

This frontend PR is related to the development backend PR. To test this frontend PR, you need to checkout the development backend PR.

Main changes explained:

  • Update src/components/CommunityPortal/CPDashboard.jsx to add date restriction functionality

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 the Community Portal: /communityportal/.
  6. Go to the "All Events" page
  7. In the "Search Filters" section on the left, locate the date input field (below "Tomorrow" and "This Weekend" radio buttons)
  8. Click on the date input field to open the calendar picker
  9. Verify: Past dates (dates before today) are disabled/grayed out and cannot be selected
  10. Verify: Today's date and future dates are selectable and highlighted

Screenshots or videos of changes:

Before :
Screenshot 2025-11-07 at 1 31 24 AM

After
Screenshot 2025-11-07 at 1 24 24 AM

@netlify
Copy link
Copy Markdown

netlify bot commented Nov 6, 2025

Deploy Preview for highestgoodnetwork-dev ready!

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

@pavanputti pavanputti added the do not review Do not review or look at code without full context label Nov 6, 2025
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Nov 8, 2025

@pavanputti pavanputti added Medium Priority and removed do not review Do not review or look at code without full context labels Nov 9, 2025
Prem203
Prem203 previously approved these changes Dec 7, 2025
Copy link
Copy Markdown
Contributor

@Prem203 Prem203 left a comment

Choose a reason for hiding this comment

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

Reviewed the PR with an admin account and the date picker does not allow selection of past dates.

image

@sonarqubecloud
Copy link
Copy Markdown

@pavanputti pavanputti changed the title Restrict past date selection in event search calendar Siva - Restrict past date selection in event search calendar Feb 1, 2026
@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 8, 2026
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,

I tested this PR locally. Past dates are disabled correctly, but I’m not able to select today’s date. Could you please check if this is expected?
PR 4338 - 1

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 Siva,

I have reviewed your PR locally and though past dates are disabled as per requirement. I am unable to select today's date as it too is disabled.

Image Image

@pavanputti
Copy link
Copy Markdown
Contributor Author

pavanputti commented Mar 14, 2026

Hi,

I tested this PR locally. Past dates are disabled correctly, but I’m not able to select today’s date. Could you please check if this is expected? PR 4338 - 1

Hi @HemanthNidamanuru

Thanks for pointing it out. I’ve fixed the date logic so today’s date is now selectable.

@pavanputti
Copy link
Copy Markdown
Contributor Author

Hi Siva,

I have reviewed your PR locally and though past dates are disabled as per requirement. I am unable to select today's date as it too is disabled.

Image Image

Hi @Anusha-Gali Anusha,

Thanks for pointing it out. I’ve fixed the date logic so today’s date is now selectable.

Anusha-Gali
Anusha-Gali previously approved these changes Mar 17, 2026
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 Siva,

Thanks for fixing the issue, it works as per requirement now.

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,

I ran the PR locally. The past dates are correctly disabled, which works as expected. However, for today’s date, no data is showing in the calendar/events.

Image Image

- Add minDate={new Date()} to CommunityCalendar ReactCalendar to disable past date selection
- Add min attribute (today) to CPDashboard date filter input; wire showPicker() so clicking the chevron opens the calendar
- Add min attribute (today) to ActivityList date filter input
- Add min attribute (today) to RegistrationForm event date input
- Add min attribute (today) to CreateEventModal event date input
- Convert CommentSection.css to CommentSection.module.css and update classNames
- Convert reportsPage.css to reportsPage.module.css; update Reports.jsx, ReportTableSearchPanel.jsx, and AddLostTime.jsx to use CSS Modules
@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 62f8237 into development Apr 12, 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.

6 participants