Skip to content

SaiKrishna_EditableAndSelectableFieldsInEquipmentBackEnd#2054

Open
saikrishnaraoyadagiri wants to merge 1 commit intodevelopmentfrom
SaiKrishna_EditableAndSelectableFieldsInEquipmentPage
Open

SaiKrishna_EditableAndSelectableFieldsInEquipmentBackEnd#2054
saikrishnaraoyadagiri wants to merge 1 commit intodevelopmentfrom
SaiKrishna_EditableAndSelectableFieldsInEquipmentPage

Conversation

@saikrishnaraoyadagiri
Copy link

@saikrishnaraoyadagiri saikrishnaraoyadagiri commented Feb 18, 2026

image

Related PRS (if any):

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

Main changes explained:

Update src/controllers/bmdashboard/bmEquipmentController.js for adding the updateEquipmentById PUT handler
Update src/models/bmdashboard/buildingInventoryItem.js for adding equipmentClass and currentUsage fields
Update src/models/bmdashboard/buildingEquipment.js for adding equipmentClass, currentUsage
Update src/routes/bmdashboard/bmEquipmentRouter.js for adding a PUT route on /equipment/:equipmentId to handle equipment updates.
Update src/models/bmdashboard/buildingMaterial.js for updating the itemType ref to invTypeBase.
Update src/models/bmdashboard/buildingTool.js for updating the itemType ref to invTypeBase…

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. Follow the instructions from the frontend PR to test this backend in the UI directly

Screenshots or videos of changes:

editableAndSelectableFieldsInEquipment.mp4

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 Sai Krishna,

I have reviewed your PR locally and the API works as per requirement.
Image
Image

Copy link

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Sai Krishna,

The backend functionalities are working as expected. The update APIs are functioning correctly, and the data is being saved and retrieved properly without any issues.

Everything within the scope of this backend PR is working fine. Approved.

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.

3 participants

Comments