Production Release: SmartICU Hotfixes #5727
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHAT
🤖 Generated by Copilot at 66a14c0
This pull request fixes some UI and validation issues in the auth, facility, and vitals monitor components. It also refactors some common components to separate files for better code organization and reusability. The files affected are
Login.tsx
,ResetPassword.tsx
,UpdateFacilityMiddleware.tsx
,UserAdd.tsx
, andHL7PatientVitalsMonitor.tsx
.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at 66a14c0
field_required
(link)required
attribute from the password field in the login component (link)LegacyErrorHelperText
from the reset password component (link)validateRule
andvalidatePassword
to the reset password component to implement the new password validation logic and UI (link)passReg
andsetPassReg
withpasswordInputInFocus
andconfirmPasswordInputInFocus
to track the focus state of the password and confirm password fields in the reset password component (link)validatePassword
in the reset password component (link)CollapseV2
component with a conditional rendering of thevalidateRule
component to show the password requirements in the reset password component, and add theonFocus
andonBlur
handlers to the password field to update the focus state (link)LegacyErrorHelperText
component and add a conditional rendering of thevalidateRule
component to show the error for the confirm password field in the reset password component, and add theonFocus
andonBlur
handlers to the confirm password field to update the focus state (link)errors
property to the initial state of the update facility middleware component to store and display the validation errors for the middleware address field (link)Notification.Error
calls with dispatch actions that set the error in the state for the empty or invalid middleware address field in the update facility middleware component (link, link)error
prop to the input field component for the middleware address in the update facility middleware component, which takes the error from the state and displays it as a helper text (link)validateRule
component from the user add component to a separate file, as it is reused in the reset password component and could be a common component for other validation scenarios (link, link)Resp
channel in the HL7 patient vitals monitor component, and make it consistent with theSpO2
channel (link)