Skip to content

Vinay K Most Frequent Keywords#5307

Open
Vinay944924 wants to merge 4 commits into
developmentfrom
vinay_k_render_keywords
Open

Vinay K Most Frequent Keywords#5307
Vinay944924 wants to merge 4 commits into
developmentfrom
vinay_k_render_keywords

Conversation

@Vinay944924

Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-05-30 at 12 14 45 AM Screenshot 2026-05-30 at 12 15 05 AM

Related PRS (if any):

This frontend PR is related to the development backend PR.
The Previous frontend PR is PR 3418

Main changes explained:

  • Updated the Most Frequent Keywords jsx and css file making the rendering issue work.

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. Dashboard → Reports → Total Construction Summary
    or http://localhost:3000/bmdashboard/totalconstructionsummary
  6. Verify the changes in Lessons learned according to the video.

Screenshots or videos of changes:

Task.mov

Note:

Include the information the reviewers need to know.

@netlify

netlify Bot commented May 30, 2026

Copy link
Copy Markdown

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit f0ab5d2
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a303e1f45ddd10008d1dee9
😎 Deploy Preview https://deploy-preview-5307--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@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 Jun 11, 2026

@HemanthNidamanuru HemanthNidamanuru left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

I tested this PR locally. The Lessons Learned section is working correctly. However, there is excess empty space on the right side of the page, which causes the Most Frequent Keywords chart to appear cramped and clustered. Please fix the layout spacing.

Image Image Image Image

@sonarqubecloud

Copy link
Copy Markdown

@Mahitha-pasupuleti Mahitha-pasupuleti left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested the PR locally following the provided steps. The Most Frequent Keywords section is rendering correctly, and the Lessons Learned data displays as expected without any issues. Functionality works as intended. Approving PR.

Image Image Image Image Image Image

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.

4 participants