Skip to content

Sayali: 🔥 resolve SonarCloud blockers - fix invalid CSS unit and remove re…#4911

Merged
one-community merged 1 commit intodevelopmentfrom
fix/sonarcloud-blockers
Feb 27, 2026
Merged

Sayali: 🔥 resolve SonarCloud blockers - fix invalid CSS unit and remove re…#4911
one-community merged 1 commit intodevelopmentfrom
fix/sonarcloud-blockers

Conversation

@sayali-2308
Copy link
Contributor

Description

Fixes SonarCloud blockers blocking PR #4906 (Frontend Release to Main)

Related PRs:

Unblocks PR #4906

Main changes explained:

  • Fixed invalid CSS unit 14x14px in Skills.module.css L46
  • Removed redundant return null in handleTeamCodeChange catch block in WeeklySummariesReport.jsx
  • Fixed media feature range notation in Skills.module.css

How to test:

  1. Check out branch fix/sonarcloud-blockers
  2. npm install && npm run start:local
  3. Verify Weekly Summaries page loads correctly
  4. Verify Skills Profile page loads correctly
  5. Verify dark mode works

Screenshots:

N/A - SonarCloud blocker fixes only

Note:

Both blockers were pre-existing issues (10 months and 4 months old). These fixes should allow SonarCloud Quality Gate to pass on PR #4906.

@netlify
Copy link

netlify bot commented Feb 27, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 23aedb9
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69a20b5045c2520008261b12
😎 Deploy Preview https://deploy-preview-4911--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 Sayali: resolve SonarCloud blockers - fix invalid CSS unit and remove re… Sayali: 🔥 resolve SonarCloud blockers - fix invalid CSS unit and remove re… Feb 27, 2026
@one-community one-community merged commit cc34d74 into development Feb 27, 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