Skip to content

Sphurthy - fix:fixed no show rate insightsfilter controls#5300

Open
sphurthy wants to merge 8 commits into
developmentfrom
sphurthy-no-show-rate-insights-filter
Open

Sphurthy - fix:fixed no show rate insightsfilter controls#5300
sphurthy wants to merge 8 commits into
developmentfrom
sphurthy-no-show-rate-insights-filter

Conversation

@sphurthy

Copy link
Copy Markdown
Contributor

Description

image

Related PRS (if any):

None

Main changes explained:

  • Update src/components/CommunityPortal/Reports/Participation/NoShowInsights.jsx to add the missing "My Event" / "All Events" dropdown alongside the existing date filter so the No-show Rate Insights section matches Figma; defaults set to "My Event" + "This Week", filtering wired into renderStats, and the export modal/PDF filename updated to include the new scope.

How to test:

  1. Check out the branch.
  2. Run npm install and npm start.
  3. Log in as admin.
  4. Go to Community Portal → Reports → Participation → No-show rate insights.
  5. Verify two dropdowns appear: "My Event" (My Event / All Events) and "This Week" (All Time / Today / This Week / This Month), defaulted to "My Event" and "This Week".
  6. Switch each dropdown and confirm the insight bars update.
  7. Verify it works in dark mode.

Screenshots or videos of changes:

4

@netlify

netlify Bot commented May 28, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 92f63a7
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a3179c9c3eb99000875e418
😎 Deploy Preview https://deploy-preview-5300--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@one-community one-community changed the title fix:fixed no show rate insightsfilter controls Sphurthy - fix:fixed no show rate insightsfilter controls May 29, 2026

@HemanthNidamanuru HemanthNidamanuru 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.

Hi,

I tested this PR locally and all requested functionality is working as expected. However, there are unresolved merge conflicts that need to be addressed before this PR can be merged.

Image Image Image Image

DeepighaJ
DeepighaJ previously approved these changes May 30, 2026

@DeepighaJ DeepighaJ 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.

Checked out to current branch and tested PR locally.
No-show rate insights dropdown and functionlity work as expected in both light and dark modes.
Please address the merge conflicts.
Image
Image
Image
Image
Image

@sonarqubecloud

Copy link
Copy Markdown

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