Skip to content

Conversation

@Amalesh-A
Copy link
Contributor

@Amalesh-A Amalesh-A commented Jan 7, 2026

Description

This PR enables users who have the Toggle Request Bio permission to access and toggle the Bio switch on the Weekly Summaries Report page.

Related PRS (if any):

This frontend PR is related to the #1990 backend PR.
Old Frontend PR: #1831
Old Backend PR: #704

Main changes explained:

  • Added a new permission label for Toggle Request Bio.
  • Added conditional logic so that users with this permission can view and toggle the Bio switch

How to test:

  1. Check out this branch locally -> Amalesh-togglebio
  2. Install dependencies and start the app:
    • npm install
    • npm run start:local
  3. Clear site data / cache (important to avoid stale permission state).
  4. Log in as an Owner user.
  5. Navigate to:
    • Other Links → Permissions Management → Manage User Permissions
  6. Choose a user without the Toggle Request Bio permission under Reports.
  7. Click Add, scroll down, then Submit.
  8. Verify:
    • The updated user can now see the Bio toggle on:
      • Reports → Weekly Summaries Report
    • The user can toggle the Bio switch successfully.

Screenshots or videos of changes:

Final.mp4

@netlify
Copy link

netlify bot commented Jan 7, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit ece2ad2
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/696521c20d42470008aa485b
😎 Deploy Preview https://deploy-preview-4679--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.

@Amalesh-A Amalesh-A changed the title Amalesh togglebio Amalesh - Toggle Request Bio permission on Weekly Summaries Report Jan 7, 2026
@Amalesh-A Amalesh-A added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Jan 8, 2026
JuliaHa0902
JuliaHa0902 previously approved these changes Jan 9, 2026
Copy link
Contributor

@JuliaHa0902 JuliaHa0902 left a comment

Choose a reason for hiding this comment

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

I have followed the instruction to test and confirmed that the toggle request bio permission enable user to toggle the Bio announcement.

Screenshot 2026-01-09 095122

nathanah
nathanah previously approved these changes Jan 10, 2026
Copy link
Contributor

@nathanah nathanah left a comment

Choose a reason for hiding this comment

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

LGTM

@Amalesh-A Amalesh-A dismissed stale reviews from nathanah and JuliaHa0902 via ece2ad2 January 12, 2026 16:30
@sonarqubecloud
Copy link

Copy link

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Amalesh,

I have reviewed your PR locally and the frontend works as expected.
Screenshot 2026-01-12 at 7 47 02 PM
Screenshot 2026-01-12 at 7 39 03 PM
Screenshot 2026-01-12 at 7 39 07 PM
Screenshot 2026-01-12 at 7 39 10 PM
Screenshot 2026-01-12 at 7 42 06 PM
Screenshot 2026-01-12 at 7 45 08 PM
Screenshot 2026-01-12 at 7 45 25 PM

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.

5 participants