Skip to content

Conversation

@shashank-madan
Copy link
Contributor

Description

image

Related PRS (if any):

This frontend PR is related to the development branch of the backend

Main changes explained:

  • Fixed protected route to correctly allow login and access to community portal routes
  • Added online events filtering criteria and ensured it works well with other filters

How to test:

  1. check into current branch
  2. do yarn install and yarn start to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to localhost:5173/communityportal
  6. Check and uncheck the online only filter and verify that it correctly filters out events.
  7. Verify that the online only filter works with the search filter.

Screenshots or videos of changes:

Online_only_filter_working.mp4

Note:

Please note that this PR is only to fix the filtering of online events. Other filters have not been implemented yet and will not work. They are not in the scope of this PR. Dark mode styling is also not in the scope of this PR.

@netlify
Copy link

netlify bot commented Dec 28, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit cf44e1c
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69528f87ce4e1e00080a24b2
😎 Deploy Preview https://deploy-preview-4645--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.

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Dec 29, 2025
@sonarqubecloud
Copy link

Copy link

@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 Shashank,

I have reviewed your PR locally and the Online filter button works as per requirement.
Screenshot 2026-01-02 at 12 40 39 AM
Screenshot 2026-01-02 at 12 40 42 AM
Screenshot 2026-01-02 at 12 41 03 AM

Copy link

@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 Shashank,

Good job with this, implementation is functional.

PR 4645 Screenshot PR 4645 Screenshot 2

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