Skip to content

Sanjeev: Fix Force Logout When Permissions Change#4439

Merged
one-community merged 7 commits intodevelopmentfrom
Sanjeev_fix_force_logout_when_permissions_change
Dec 4, 2025
Merged

Sanjeev: Fix Force Logout When Permissions Change#4439
one-community merged 7 commits intodevelopmentfrom
Sanjeev_fix_force_logout_when_permissions_change

Conversation

@sanjeev29
Copy link
Copy Markdown
Contributor

Description

Screenshot 2025-10-29 at 12 51 19 PM

Related PRS (if any):

...

Main changes explained:

  • Fixed PermissionWatcher.jsx to distinguish mid-session permission changes from initial login with unacknowledged permissions.
  • Added STOP_FORCE_LOGOUT action in authReducer.js
  • Added stopForceLogout function in authActions.js that stops any active force logout timer and clears related state

How to test:

  1. check into current branch
  2. do npm install and npm run start:local to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. As an admin, change a user's permissions while they are logged out
  6. Log in as that user
  7. Verify: A PopUpBar appears indicating permission were changed for the user
  8. Verify: No countdown timer is displayed (banner only)
  9. Closing the PopUpBar will close acknowledge the new permissions.

Screenshots or videos of changes:

https://www.dropbox.com/scl/fi/yj6sl21a9958e13410t8l/Fix-force-logout-when-permissions-change.mov?rlkey=xkvy8a7w1napt124435e1xat9&st=guqf769v&dl=0

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 22, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 8be1ee7
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/692e49f676d9910008bd7841
😎 Deploy Preview https://deploy-preview-4439--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.

nathanah
nathanah previously approved these changes Nov 29, 2025
Copy link
Copy Markdown
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

@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 Dec 1, 2025
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 2, 2025

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit b880273 into development Dec 4, 2025
10 checks passed
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.

3 participants