Skip to content

Neeraj add event sorting by start time#4568

Merged
one-community merged 3 commits intodevelopmentfrom
Neeraj_Add_Event_Sorting_By_Start_Time
Mar 25, 2026
Merged

Neeraj add event sorting by start time#4568
one-community merged 3 commits intodevelopmentfrom
Neeraj_Add_Event_Sorting_By_Start_Time

Conversation

@Neeraj-Kondaveeti
Copy link
Copy Markdown
Contributor

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

Description

image

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

  • Added a Sort By option to the Community Portal Activities list, allowing events to be ordered by date (Earliest → Latest or Latest → Earliest).
  • Added component tests for ActivityList to cover rendering, filtering, sorting behavior, and dark-mode rendering in preparation for upcoming frontend coverage enforcement.

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/Activities
  6. Use the Sort By dropdown to switch between Earliest to Latest and Latest to Earliest and confirm the list reorders dynamically.

Screenshots or videos of changes:

image image

Note:

This change is frontend-only; no backend logic was modified.

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

netlify Bot commented Dec 16, 2025

Deploy Preview for highestgoodnetwork-dev ready!

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

@Neeraj-Kondaveeti Neeraj-Kondaveeti added High Priority and removed do not review Do not review or look at code without full context labels Dec 17, 2025
sayali-2308
sayali-2308 previously approved these changes Feb 5, 2026
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

Frontend #4568: Reviewed event sorting feature for Community Portal Activities. Checked out branch Neeraj_Add_Event_Sorting_By_Start_Time, ran npm install, cleared cache. Backend on development (frontend-only change, no backend modifications). Logged in as admin, navigated to http://localhost:5173/communityportal/Activities.

Tested Sort By functionality:

  1. "Start Time: Earliest to Latest": Events displayed in chronological order (oldest dates first: 2023-12-30, then 2024-01-10, 2024-01-12, etc.)
  2. "Start Time: Latest to Earliest": Events displayed in reverse chronological order (newest dates first: 2024-02-01, 2024-01-25, 2024-01-20, etc.)
  3. List reorders dynamically without page refresh when changing sort option
  4. Date filter compatibility: Applied date filter "30-12-2023" with "Latest to Earliest" sort - correctly displayed only events from that date in sorted order (Coding Workshop, Photography Walk both 2023-12-30)
  5. Dark mode: Sort By dropdown visible with proper contrast, activities list readable, no styling issues

Light mode: All elements properly styled and functional
Console: No errors, only standard initialization messages

Feature works exactly as described in PR. Frontend-only change successfully adds sorting capability to Activities list. All functionality tested and working correctly. APPROVED.

@beblicarl
Copy link
Copy Markdown
Contributor

This feature works as intended and the code is great

image image

@beblicarl beblicarl self-requested a review February 12, 2026 08:22
beblicarl
beblicarl previously approved these changes Feb 12, 2026
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.

Image 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
@Neeraj-Kondaveeti Neeraj-Kondaveeti force-pushed the Neeraj_Add_Event_Sorting_By_Start_Time branch from bfe5dd2 to 11f5d0c Compare March 22, 2026 21:42
@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 386e681 into development Mar 25, 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.

5 participants