Skip to content

Ramsundar - Fixed three Functionality issues on the Inventory types page(Backend)#2131

Closed
Ram-blip wants to merge 5 commits into
developmentfrom
Backend-inventory-types-functional-issue-fixes
Closed

Ramsundar - Fixed three Functionality issues on the Inventory types page(Backend)#2131
Ram-blip wants to merge 5 commits into
developmentfrom
Backend-inventory-types-functional-issue-fixes

Conversation

@Ram-blip
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-03-27 at 4 42 38 PM

Related PRS (if any):

This backend PR is related to the #5074 frontend PR.

Main changes explained:

  • Separated the reusable create flow from the tool create flow.
  • Migrated “unit of measurement” storage from the old JSON-file approach to a MongoDB model, The backend now reads, adds, and deletes units through the database.

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 "http://localhost:5173/bmdashboard/inventorytypes"
  6. First check: Go add item in reusables, items will get added in reusables, there is no need to refresh, it updates immediately. Do the same thing for tools.
  7. Second check: Go to unit of measurement, add any measurement, it will get added. Also, the delete will also work. Error page won't come.

Screenshots or videos of changes:

Video Proof After changes:
https://www.loom.com/share/c1f26c4fe96241c2b5c6cb107c69b955

Note:

Include the information the reviewers need to know.

@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 May 5, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
13.0% Duplication on New Code (required ≤ 3%)
D Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Ram-blip
Copy link
Copy Markdown
Contributor Author

Redone with this PR: #2210

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.

2 participants