Skip to content

Sundar - 🔥 Fix sonar reliability issue in logout#4920

Merged
one-community merged 1 commit intodevelopmentfrom
sundar/fix-sonar
Feb 28, 2026
Merged

Sundar - 🔥 Fix sonar reliability issue in logout#4920
one-community merged 1 commit intodevelopmentfrom
sundar/fix-sonar

Conversation

@sundarmachani
Copy link
Contributor

@sundarmachani sundarmachani commented Feb 28, 2026

Description

This PR fixes a SonarCloud reliability issue that was causing the Quality Gate to fail and blocking the release merge from development → main.

The issue was caused by an unused JSX expression inside the logout handler (Logout.jsx) which SonarCloud identified as a reliability bug.

Related PRS (if any):

NA

Main changes explained:

  • Removed unused JSX expression inside Logout.jsx.
  • Ensured logout navigation is handled correctly using programmatic routing.
  • Resolved SonarCloud “Expected an assignment or function call and instead saw an expression” issue.
  • Restored SonarCloud Quality Gate compliance (Reliability Rating → A).

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

Screenshots or videos of changes:

Note:

NA

@netlify
Copy link

netlify bot commented Feb 28, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit d522e3b
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69a29aa660bd250008d7daad
😎 Deploy Preview https://deploy-preview-4920--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

@one-community one-community changed the title Fix sonar reliability issue in logout Sundar - 🔥 Fix sonar reliability issue in logout Feb 28, 2026
@one-community one-community merged commit 4af1eb2 into development Feb 28, 2026
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