Skip to content

Julia - Fix the error in development branch when user permission change, they cannot login to their account.#4501

Merged
one-community merged 1 commit intodevelopmentfrom
julia-fix-isAckLoading-missing
Dec 5, 2025
Merged

Julia - Fix the error in development branch when user permission change, they cannot login to their account.#4501
one-community merged 1 commit intodevelopmentfrom
julia-fix-isAckLoading-missing

Conversation

@JuliaHa0902
Copy link
Copy Markdown
Contributor

Description

Fix the error in development branch when user permission change, they cannot login to their account.

Related PRS (if any):

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

Main changes explained:

  • Add back this line in Header: const [isAckLoading, setIsAckLoading] = useState(false);

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 admin user
  5. go to dashboard -> Other Links -> Permission Management
  6. Choose a Volunteer and change their permission
  7. Login as the Volunteer
  8. Confirm that you are able to login with no error

Screenshots or videos of changes:

Screen.Recording.2025-12-05.004402.mp4

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 5, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 2669548
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69327f8a1f2410000834739b
😎 Deploy Preview https://deploy-preview-4501--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

sonarqubecloud Bot commented Dec 5, 2025

@one-community one-community merged commit 8d0d872 into development Dec 5, 2025
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