Skip to content

Amalesh - Toggle Request Bio permission on Weekly Summaries Report#1990

Merged
one-community merged 9 commits intodevelopmentfrom
Amalesh-togglebio
Mar 8, 2026
Merged

Amalesh - Toggle Request Bio permission on Weekly Summaries Report#1990
one-community merged 9 commits intodevelopmentfrom
Amalesh-togglebio

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 backend PR is related to the #4679 frontend 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

@Amalesh-A Amalesh-A changed the title Amalesh - fix(bio): solve toggle bio request issues Amalesh - Toggle Request Bio permission on Weekly Summaries Report Jan 7, 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

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 have mentioned my comments in frontend. The PR does work as expected.

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.

These changes don't seem related to the Request Bio permission. They seem to be related to the 'addDeleteEditOwners' permission.

Copy link

@HemanthNidamanuru HemanthNidamanuru 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 tested this PR locally and everything in the backend is working as expected.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 8, 2026

@one-community
Copy link
Member

Thank you all, merging!

@one-community one-community merged commit 201219a into development Mar 8, 2026
3 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.

6 participants