Skip to content

Ramakrishna fix dashboard view#4021

Closed
Ramaruva wants to merge 3 commits intodevelopmentfrom
Ramakrishna_Fix_dashboard_view
Closed

Ramakrishna fix dashboard view#4021
Ramaruva wants to merge 3 commits intodevelopmentfrom
Ramakrishna_Fix_dashboard_view

Conversation

@Ramaruva
Copy link
Copy Markdown
Contributor

@Ramaruva Ramaruva commented Sep 5, 2025

Description

image image

Or Implements # (WBS)

Related PRS (if any):

To test This PR please use latest backend development branch.

Main changes explained:

-Updated the header.jsx component with proper func to enhance header view for viewing user.

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/owner user
  5. go to dashboard→ leaderboard
  6. Now click on red/green status bar of any volunteer user
  7. Now it will replace the authenticated user with the respective volunteer user.
  8. Now verify that you are able to see the volunteer user header only (for clarity please login in diff browser or incognito with the same volunteer account and compare the both headers in ur dashboard and another dashboard).
  9. Now can please use other then admin/owner account to perform above steps except now you shouldn't able to interact with header icons with anything.

@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 5, 2025

Deploy Preview for highestgoodnetwork-dev failed.

Name Link
🔨 Latest commit 9a96d7a
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69277a7c8997fd0008cd55e9

@sonarqubecloud
Copy link
Copy Markdown

@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 Nov 7, 2025
@Anusha-Gali Anusha-Gali self-requested a review November 7, 2025 22:51
Copy link
Copy Markdown

@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 Ramakrishna,

I have tested this PR locally. But faced an issue when i try to use the functionality as an "Admin". Could you please check on the Admin role

Admin - Doesn't work as per the functionality i.e i am not able to see the header dashboard
Screenshot 2025-11-07 at 5 49 56 PM
Screenshot 2025-11-07 at 5 50 51 PM

Owner - Works as per the functionality i.e able to see the header dashboard
Screenshot 2025-11-07 at 5 44 53 PM
Screenshot 2025-11-07 at 5 46 11 PM
Screenshot 2025-11-07 at 5 46 19 PM

Volunteer, Mentor, Core Team, Manager - Works as per the functionality i.e to not be able to see the header dashboard
Screenshot 2025-11-07 at 5 37 09 PM

Anusha-Gali
Anusha-Gali previously approved these changes Nov 11, 2025
Copy link
Copy Markdown

@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 RamaKrishna,

I have tested your application again, the functionality does work for Admin's as well. I was getting the said permission error since i was trying to see the header of an owner account but i suppose that is blocked on purpose.
Access blocked for owner account
Screenshot 2025-11-11 at 12 55 02 AM
Access granted for non-owner account
Screenshot 2025-11-11 at 12 57 42 AM
Screenshot 2025-11-11 at 12 58 23 AM

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
5 Security Hotspots
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Ramaruva
Copy link
Copy Markdown
Contributor Author

Ramaruva commented Dec 7, 2025

HI @one-community , @Anusha-Gali , I don't know how this PR got this many file changes and this many merge conflicts so instead of resolving this many merge conflicts , I have just created a new pr since my core logic is to change only one file i.e header.jsx and here is the updated pr
4521

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