Skip to content

Prem - Date-Based Functionality Not Working#4305

Open
Prem203 wants to merge 4 commits intodevelopmentfrom
Prem-Date-functionality-not-working
Open

Prem - Date-Based Functionality Not Working#4305
Prem203 wants to merge 4 commits intodevelopmentfrom
Prem-Date-functionality-not-working

Conversation

@Prem203
Copy link
Copy Markdown
Contributor

@Prem203 Prem203 commented Nov 1, 2025

Description

image

Related PRS (if any):

This frontend PR is not related to any backend PR.

Main changes explained:

  • Update file CPDashboard to include date based filter. Also added functionality for tomorrow and this weekend radio buttons so that users can now filter data using any of the three choices. Modified the mock data so that PR can be tested by using the radio buttons.

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 /communityportal
  6. verify this weekend radio button displays data for coming weekend.
  7. verify tomorrow radio button displays events for tomorrow.
  8. verify date filter filters out the events happening on exact date.

Screenshots or videos of changes:

image image image

Note:

The dark mode is not included in the task description and hence was not implemented. Only check the date picker and the two radio buttons.

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 1, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 26e941b
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6935c6b29acb760008cf445e
😎 Deploy Preview https://deploy-preview-4305--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.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Nov 1, 2025

@Prem203 Prem203 added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Nov 1, 2025
Copy link
Copy Markdown
Contributor

@bhanuanishakkineni bhanuanishakkineni left a comment

Choose a reason for hiding this comment

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

Hello Prem,
I have reviewed this PR and tested the functionality of filtering events based on dates.
I tested this using all three that is, tomorrow radio button, this weekend radio button and custom date filter. For this I had to update the dates on mock events from CPDashboard.jsx
The functionality is working as expected. This is able to filter PRs based on the selected date.

PR.4305.testing.mp4

@one-community one-community changed the title Date-Based Functionality Not Working Prem - Date-Based Functionality Not Working Dec 7, 2025
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 7, 2025

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.

Functionality working as expected.

PR 4305 Screenshot PR 4305 Screenshot 2

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

I have reviewed your PR locally and the main requirements of functionality does work as expected but i did notice that the view of the page is different than that of your as shown in the video. Please have a look into it.
Screenshot 2025-12-27 at 1 43 20 AM
Screenshot 2025-12-27 at 1 43 25 AM
Screenshot 2025-12-27 at 1 43 37 AM
Screenshot 2025-12-27 at 1 44 39 AM

Copy link
Copy Markdown
Contributor

@sayali-2308 sayali-2308 left a comment

Choose a reason for hiding this comment

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

Image
  • Branch: Prem-Date-functionality-not-working
  • Tested as: Admin user
  • Browser: Chrome
  • Date: January 29, 2026

What Was Tested:

  1. Tomorrow Filter:

    • Clicked "Tomorrow" radio button
    • Date auto-filled to 29-01-2026 (tomorrow's date)
    • Events filtered correctly - showed "No events available" for tomorrow
    • Filter logic working as expected
  2. This Weekend Filter:

    • Clicked "This Weekend" radio button
    • Events filtered to weekend dates
  3. Date Picker:

    • Used date picker to select specific dates
    • Events filtered correctly based on selected date
  4. Default View:

    • All Events displays upcoming events correctly
    • Event cards show proper information (date, time, location)

Minor CSS/Styling Observations:

  • Dark mode styling could be improved (contrast/readability)
  • Some CSS alignment issues noticed in Search Filters section
  • These are cosmetic issues and don't affect functionality

Date-based filtering functionality works correctly. Tomorrow, This Weekend, and date picker filters all function as expected. Minor CSS improvements could be made in future iterations, but core functionality is solid.

@one-community one-community added Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. do not review Do not review or look at code without full context and removed High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible labels Mar 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants