Skip to content

Conversation

@SudheeshTD
Copy link
Contributor

@SudheeshTD SudheeshTD commented Nov 15, 2025

Description

image image image image image

Related PRS (if any):

This Backend PR is related to the #4401 Frontend PR.
To test this backend PR you need to checkout this and the Frontend PR #4401

Main changes explained:

  • Created Controllers for Materials Utilization, retrieve project name and materials name.

How to test:

  1. check into current branch and the frontend branch #4401.
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to "/materials/utilization"
  6. verify Data is visible in chart.
  7. Try using different filters and test how it behaves.

Screenshots or videos of changes:

image image image

Note:

Include the information the reviewers need to know.

@SudheeshTD SudheeshTD marked this pull request as ready for review November 20, 2025 19:00
@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 27, 2025
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 Sudeesh,

I have reviewed your PR locally and the backend API call returns response as per the requirement except for the total Handled value.
Screenshot 2025-11-27 at 1 59 41 AM

Copy link

@rohanrastogi311 rohanrastogi311 left a comment

Choose a reason for hiding this comment

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

Hi Sudheesh, I'm having a filter load error with no graphs showing. Happy to review again, once error is corrected.

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.

5 participants