Skip to content

Ramsundar - Resolved Page Crash for Bar Chart For Lessons Learned#4845

Open
Ram-blip wants to merge 1 commit intodevelopmentfrom
Fixed_Page_Crash_in_LessonsLearntPage
Open

Ramsundar - Resolved Page Crash for Bar Chart For Lessons Learned#4845
Ram-blip wants to merge 1 commit intodevelopmentfrom
Fixed_Page_Crash_in_LessonsLearntPage

Conversation

@Ram-blip
Copy link
Contributor

Description

Screenshot 2025-12-06 at 1 02 36 PM Screenshot 2025-12-06 at 1 02 49 PM

Related PRS (if any):

This frontend PR is related to the latest development branch.

Main changes explained:

  • The main problem the LessonsLearntChart page was not fetching anything when you tried to access it because the frontend code was written with incorrect assumptions about how to communicate with the backend. I’m not fully sure what the page is supposed to do overall. I confirmed with the person who assigned the task that my goal was only to make the page work and fetch the correct data.

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 this page "http://localhost:5173/LessonsLearntChart", Check that everything works properly.

Screenshots or videos of changes:

https://www.loom.com/share/75d9794e870f408cbaec211b59ec2ffb

Note:

Include the information the reviewers need to know.

@netlify
Copy link

netlify bot commented Feb 14, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 4f176d4
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69904f8e968aa9000868d2a6
😎 Deploy Preview https://deploy-preview-4845--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 link

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

I have reviewed your PR locally and the page does load without any crash out, however there are quite a few issues with the page which i assume is not part of this PR.

Image Image

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

Comments