Skip to content

Aayush taking over for Vamsi krishna creates a stacked horizontal bar graph for tools stoppage reason#1550

Open
vamsikrishna1704 wants to merge 9 commits into
developmentfrom
Vamsi_Krishna_Create_a_stacked_horizontal_bar_graph_for_tools_stoppage_reason
Open

Aayush taking over for Vamsi krishna creates a stacked horizontal bar graph for tools stoppage reason#1550
vamsikrishna1704 wants to merge 9 commits into
developmentfrom
Vamsi_Krishna_Create_a_stacked_horizontal_bar_graph_for_tools_stoppage_reason

Conversation

@vamsikrishna1704
Copy link
Copy Markdown
Contributor

Description

image

Related PRS (if any):

This backend PR is related to the 3780 frontend PR.

Main changes explained:

  • Create bmToolsStoppageReasonController.js file for defining the functions to fetch unique projects and tools details.
  • Create bmToolsStoppageReasonRouter.js file to map the uri to associated controller function.
  • Create buildingToolStoppage.js file to define the schema model in DB.

How to test:

  1. check into current branch
  2. do npm install and npm run build then npm start to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to dashboard→ Reports→ Total Construction Summary→Tools and Equipment Tracking.
  6. verify filters to see appropriate results.

Screenshots or videos of changes:

ReasonOfStoppageOfTools-walkthrough.mp4

@beblicarl
Copy link
Copy Markdown
Contributor

Tested the frontend of this and approved it because everything works as intended

https://www.loom.com/share/f7064847b0e6449490b6c5c527584057

@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 Sep 20, 2025
laynet
laynet previously requested changes Sep 24, 2025
Copy link
Copy Markdown

@laynet laynet left a comment

Choose a reason for hiding this comment

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

I can't test this through the local host because the frontend branch can't be resolved, I submitted a review for that OneCommunityGlobal/HighestGoodNetworkApp#3780 (review)
PR 3780

Copy link
Copy Markdown

@MeronTeweldebrhan MeronTeweldebrhan left a comment

Choose a reason for hiding this comment

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

PR Review Feedback

Testing Outcome

✅ Verified Tools and Equipment Tracking Card under Dashboard → Reports → Total Construction Summary.
✅ Proportion of Tools/Equipment filter works as expected.
✅ Tools by Availability chart functions correctly.
✅ Dark mode verified.
❌ Reason of Stoppage of Tools fails to load, showing: “Failed to load projects. Please try again.”

Suggestion
In the Proportion of Tools/Equipment filter, after selecting a project, switching directly to another project doesn’t refresh the list unless “All” is clicked first. Improving this behavior would enhance user experience.

Screenshot 2025-09-25 112806 Screenshot 2025-09-25 112823

@vamsikrishna1704
Copy link
Copy Markdown
Contributor Author

Hi @laynet & @MeronTeweldebrhan, the problems you are facing is due to the recent node upgrade. I have resolved the node upgrade conflicts. Try to check the changes which address the errors you are facing. Thank you.

@dunstanad
Copy link
Copy Markdown

I tested the backend using frontend #3780 but the code didn't work as expected.

error

@sanjeev29
Copy link
Copy Markdown

Tested the changes with the related frontend PR - 3780. The filters and bar graph works as expected.
PR-3780+1550

@sumedhkumar96 sumedhkumar96 requested review from sumedhkumar96 and removed request for MeronTeweldebrhan October 10, 2025 03:41
Copy link
Copy Markdown
Contributor

@sumedhkumar96 sumedhkumar96 left a comment

Choose a reason for hiding this comment

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

Pulled the backend code and tested it along with the frontend locally, and functionalities are working as described by the dev. Screenshots are pinned in the frontend PR-3780

@vamsikrishna1704 vamsikrishna1704 dismissed stale reviews from laynet and MeronTeweldebrhan October 11, 2025 19:02

I have resolved the fetch error due to recent node upgrade.

@apoorvajainrp21
Copy link
Copy Markdown
Contributor

the projects are not loading:
Screenshot 2025-10-17 114344

@sumedhkumar96 sumedhkumar96 added Do Not Review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. and removed High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible Do Not Review Do not review or look at code without full context labels Oct 18, 2025
@Vinay944924
Copy link
Copy Markdown

Hi Vamsi,
I tested the PR locally and given the explanation in the frontend PR

@Sriamshreddy000
Copy link
Copy Markdown

Tested this PR along with the related frontend PR #3780.
Left detailed feedback and screenshots in the frontend PR for reference.

@one-community one-community changed the title Vamsi krishna creates a stacked horizontal bar graph for tools stoppage reason Aayush taking over for Vamsi krishna creates a stacked horizontal bar graph for tools stoppage reason Apr 12, 2026
…eason_backend

Mani shashank fix: add tools stoppage reason API endpoints
@one-community one-community added High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible and removed Do Not Review Do not review or look at code without full context Needs New Developer This is a PR that is partially developed but needs someone new to take it over and finish it. labels May 11, 2026
@sonarqubecloud
Copy link
Copy Markdown

@DeepighaJ
Copy link
Copy Markdown

I tested the PR locally and given the feedback in the frontend PR 4508

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.