Skip to content

Manoj - Finish fixing so tasks of members are hidden with the teams toggle. #5381

Open
ManojPuttaswamy wants to merge 4 commits into
developmentfrom
manoj_teams
Open

Manoj - Finish fixing so tasks of members are hidden with the teams toggle. #5381
ManojPuttaswamy wants to merge 4 commits into
developmentfrom
manoj_teams

Conversation

@ManojPuttaswamy

@ManojPuttaswamy ManojPuttaswamy commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

WhatsApp Image 2026-06-06 at 23 29 59

Related PRS (if any):

This frontend PR is related to the #2269 backend PR.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as Owner/admin user
  5. go to dashboard→ other links→ teams→…
  6. open member modal and verify the toggle functionality
  7. verify that the selection is persistent and is saved into the database.
  8. verify the functionality by logging in as different users.

Screenshots or videos of changes:

REC-20260709173335.mp4

@netlify

netlify Bot commented Jul 9, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 2724575
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a5022455cc8600008fcabb6
😎 Deploy Preview https://deploy-preview-5381--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.

@sonarqubecloud

sonarqubecloud Bot commented Jul 9, 2026

Copy link
Copy Markdown

@ManojPuttaswamy ManojPuttaswamy changed the title Manoj teams Manoj - Finish fixing so tasks of members are hidden with the teams toggle. Jul 11, 2026
@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jul 11, 2026

@sundarmachani sundarmachani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I’ve tested this PR along with the Backend PR 2269, and it’s working as expected. Well done on the implementation!

Screen.Recording.2026-07-11.at.10.23.45.PM.mov

@shree-vaths shree-vaths 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.

Hello, I tested #5381 on my local and its working as expected.

Passing test cases
-The admin is able to flip the toggle on and off.
-The toggle state remains off after closing and reopening the modal.
-When turned off, the volunteer user is not able to see the team on his/her profile under teams tab.
-The functionality works same in light and dark mode.

Screen.Recording.2026-07-15.at.11.00.02.AM.mov

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

Hello, I have tested this PR along with #2269 on the backend. It is working as expected.

Screen.Recording.2026-07-17.at.15.58.19.mov

@AnshShah-18 AnshShah-18 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.

Tested frontend PR #5381 locally with backend PR #2269 using Administrator and Volunteer accounts. I confirmed that the visibility toggle works correctly, remains persistent after reopening the modal and refreshing the page, and properly hides or restores the team for the Volunteer account. The frontend functionality is working as expected. Testing video attached.

PR1.mp4

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

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants