Skip to content

Use stable event listing dependencies - #10606

Open
Tomeshwari-02 wants to merge 1 commit into
SandeepVashishtha:masterfrom
Tomeshwari-02:fix/10596-events-page-listing-deps
Open

Use stable event listing dependencies#10606
Tomeshwari-02 wants to merge 1 commit into
SandeepVashishtha:masterfrom
Tomeshwari-02:fix/10596-events-page-listing-deps

Conversation

@Tomeshwari-02

Copy link
Copy Markdown

Summary

  • destructure the event listing setters used inside EventsPage effects
  • include concrete setter dependencies instead of relying on the whole listing object
  • make setSafePage stable and clean up the adjacent advanced filter callback dependency

Issue

Closes #10596

Validation

  • npx eslint src\Pages\Events\EventsPage.js src\Pages\Events\useEventListing.js --max-warnings=0
  • node --test tests\routeSearchUtils.test.mjs tests\eventPaginationUtils.test.mjs

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

@Tomeshwari-02 is attempting to deploy a commit to the sandeepvashishtha's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added quality:exceptional 1.5x gssoc:approved gssoc approved PR level:intermediate 35pts type:bug type:devops CI/CD, infrastructure, or tooling changes size/M Medium pull request labels Jul 14, 2026

@codescene-access codescene-access Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Our agent can fix these. Install it.

No quality gates enabled for this code.

Quality Gate Profile: Custom Configuration
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved gssoc approved PR level:intermediate 35pts quality:exceptional 1.5x size/M Medium pull request type:bug type:devops CI/CD, infrastructure, or tooling changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: EventsPage effects use listing methods without stable dependencies

1 participant