Fixed #15600: Adds extra verification step on checkin of device if enabled#18538
Fixed #15600: Adds extra verification step on checkin of device if enabled#18538Glukose1 wants to merge 1 commit intogrokability:developfrom
Conversation
|
I'm not sure we'd want this at a settings level though? Also why couldn't you use the checkbox in the required custom fields on checkin/checkout? |
|
The required custom field would work; however, I would need to ensure that every asset model where I want the additional verification has the fieldset assigned. For existing models, this is not a problem, but for newly created models, I see it being one. Especially when multiple people, with varying levels of experience, are using the system. Small things like this can easily be forgotten or overlooked in a hurry. I personally prefer the simple solution of making it a global setting that does not need to be maintained by the user, as the custom fields would. |
Fixes #15600
This PR implements a modal that appears when the check-in button is clicked on the asset's check-in page. The user must click a checkbox before they can check in the asset. This behaviour can be enabled or disabled in the general settings tab. The modal has two text fields. One for a general text message in the modal and one next to the checkbox. The text for these fields can also be defined on the general settings page.
Setting disabled:

Setting enabled, with text already entered:

New verification modal after clicking check-in on the asset page, the asset cannot be checked in:

Verification modal after checking the checkbox, the asset can now be checked in:

Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist: