Skip to content

Chirag - Fix: Added logic to return all events for community calendar#2198

Merged
one-community merged 2 commits into
developmentfrom
chirag-fix-event-display-on-community-calendar
May 24, 2026
Merged

Chirag - Fix: Added logic to return all events for community calendar#2198
one-community merged 2 commits into
developmentfrom
chirag-fix-event-display-on-community-calendar

Conversation

@ChiragBellara
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-05-05 at 12 15 59 PM

Related PRS (if any):

This is related to

Main changes explained:

  • Updated the GET /events to return all events in default scenarios and paginated results when page and limit are specified.

How to test:

  1. check into current branch
  2. do npm install and npm run dev to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. using Postman try hitting the http://localhost:4500/api/events -> you should see all the events at once
  6. again try hitting http://localhost:4500/api/events?page=1&limit=10 -> you should see only the first 10 events.

Screenshots or videos of changes:

GET default scenario Screenshot 2026-05-05 at 12 18 12 PM

Copy link
Copy Markdown

@DeepighaJ DeepighaJ left a comment

Choose a reason for hiding this comment

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

Checked into current branch and ran PR locally
In Postman created a get request for the endpoint http://localhost:4500/api/events and got 200 response status with all events and its data in response body.
Validated a get request for the endpoint http://localhost:4500/api/events?page=1&limit=10 and got 200 response status with 10 events retrieved in response body.

Image Image

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 326c438 into development May 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants