Skip to content

Ramsundar - Backend Fix "Usage Record" modal not displaying Data#1921

Closed
Ram-blip wants to merge 1 commit intodevelopmentfrom
Backend_Fix_Usage_Record_Modal_Not_displaying_Data
Closed

Ramsundar - Backend Fix "Usage Record" modal not displaying Data#1921
Ram-blip wants to merge 1 commit intodevelopmentfrom
Backend_Fix_Usage_Record_Modal_Not_displaying_Data

Conversation

@Ram-blip
Copy link
Copy Markdown
Contributor

@Ram-blip Ram-blip commented Nov 25, 2025

Description

Screenshot 2025-11-24 at 11 02 48 PM

Related PRS (if any):

To test this backend PR you need to checkout the #4455 frontend PR.

Main changes explained:

  • Email was not displaying, Fixed it.

How to test:

  1. check into current branch
  2. do npm install and npm run start:local to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to /bmdashboard/consumables
  6. Click the View button under usage Record Column and check all the records

Screenshots or videos of changes:

https://www.loom.com/share/398669df8e984df28cbd3dd5f919b856

Note:

Include the information the reviewers need to know.

Copy link
Copy Markdown

@VijayAnirudh VijayAnirudh left a comment

Choose a reason for hiding this comment

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

Please refer to the comments made here: OneCommunityGlobal/HighestGoodNetworkApp#4455 (review)

@Ram-blip
Copy link
Copy Markdown
Contributor Author

Please ensure that createdBy.email and requestedBy.email are correctly populated from the backend so the modal displays complete data.

Hey Vijay, thanks for testing and for the detailed notes.
Can you try testing with a production build (npm run build then npm start) and re-check? Because it is working fine for me.

Screenshot 2025-12-20 at 12 23 41 AM

@Ram-blip
Copy link
Copy Markdown
Contributor Author

I checked the latest dev website and it looks like someone has worked on it. Last time the page was connected to a different endpoint. I think someone created a new endpoint and changed the UI, so everything is different now. So, I am closing this PR.

@Ram-blip Ram-blip closed this Feb 18, 2026
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