Skip to content

Diya 🔥 fix(wspage): Fixed Weekly Summary Page Issues#5149

Merged
one-community merged 1 commit intodevelopmentfrom
Diya_Fix_WeeklySummaryPage
Apr 15, 2026
Merged

Diya 🔥 fix(wspage): Fixed Weekly Summary Page Issues#5149
one-community merged 1 commit intodevelopmentfrom
Diya_Fix_WeeklySummaryPage

Conversation

@DiyaWadhwani
Copy link
Copy Markdown
Contributor

Description

Fixes two regressions on the Weekly Summaries Report page where the Manage Filters button and the Emails section at the bottom were both missing for Owner/Administrator users.

Related PRs (if any):

None

Main changes explained:

  • Updated WeeklySummariesReport.jsx to add canManageFilter to the fetchInitialPermissions useEffect — it was missing from the permission state initialization that runs on mount, causing the Manage Filters button to never appear even for Owners and Administrators
  • Updated WeeklySummariesReport.jsx to pass auth={props.auth} instead of auth={state.auth} to FormattedReportstate.auth was always the initial empty value [] since the function that set it was never called, causing the Emails section to always return null

How to test:

  1. Check out the current branch
  2. Run npm install --force and start the dev server with npm run start:local
  3. Clear site data/cache
  4. Log in as an Owner or Administrator
  5. Navigate to Reports → Weekly Summaries
  6. Verify the Manage Filters button is visible in the top right next to Select Filter
  7. Click Manage Filters and verify that the "Create New Filter" and "Update/Delete" Filter options appear
  8. Select any team code to filter users
  9. Scroll to the bottom of the results and verify the Emails section appears with the mail icon and copy button, showing the filtered users' emails

Screenshots or videos of changes:

Before:
Screenshot 2026-04-14 at 6 11 54 PM

After:
Screenshot 2026-04-14 at 6 10 02 PM

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 15, 2026

Deploy Preview for highestgoodnetwork-dev ready!

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

@one-community one-community changed the title Diya fix(wspage): Fixed Weekly Summary Page Issues Diya 🔥 fix(wspage): Fixed Weekly Summary Page Issues Apr 15, 2026
@one-community one-community merged commit c4a72a3 into development Apr 15, 2026
10 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.

2 participants