Skip to content

Conversation

@JuliaHa0902
Copy link
Contributor

@JuliaHa0902 JuliaHa0902 commented Dec 26, 2025

Description

Combine the functionality of the mini filter and the big filter for weekly summaries.

Related PRS (if any):

This frontend PR is related to the #1976 backend PR.
To test this frontend PR you need to checkout the #1976 backend PR.

Main changes explained:

  • Move API logic to weeklySummariesFilterAction
  • Move all the filter modals to the component folder
  • Connect the frontend of the mini-filter with the API logic in weeklySummariesFilterAction
  • Add filter API logic to UserTeamTable and TeamTable

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. check the backend into the backend PR #1976
  4. do npm install, npm run build and ... to run the backend PR locally
  5. Clear site data/cache
  6. log as admin/owner user
  7. go to report > Weekly Summaries Reports
  8. Test the mini filter function in the Select Team Code. Verify that the mini filter function works and identical with the filter in Select Filter button and Manage Filter button

Screenshots or videos of changes:

PR.test.guide.mp4

Note:

Include the information the reviewers need to know.

@netlify
Copy link

netlify bot commented Dec 26, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 52e3354
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/694ef92bdae7f10008b3ca85
😎 Deploy Preview https://deploy-preview-4629--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

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