Closed ajhollid closed 5 days ago
[!CAUTION]
Review failed
The pull request is closed.
The pull request introduces several modifications across multiple components in the client application. Key changes include the replacement of the Field
component with the TextInput
component in various registration steps, enhancing input handling and error reporting. The Check
component's error color handling has been adjusted, and a new optional prop, noHighlightText
, has been added. Additionally, the TextInput
component has been updated to accept new props for improved functionality and styling, including adjustments to helper text positioning.
File | Change Summary |
---|---|
Client/src/Components/Check/Check.jsx |
Updated error color handling from theme.palette.error.contrastText to theme.palette.error.main . Added noHighlightText prop. Updated prop types. |
Client/src/Components/Inputs/TextInput/index.jsx |
Added props: id (required), name , onBlur , disabled . Updated styles for helper text. Updated prop types. |
Client/src/Pages/Auth/Register/StepOne/index.jsx |
Replaced Field with TextInput . Added helperText prop for error messages. Adjusted error handling logic. |
Client/src/Pages/Auth/Register/StepTwo/index.jsx |
Replaced Field with TextInput for email input. Updated error handling logic and added helperText prop. |
Client/src/Pages/Auth/Register/StepThree/index.jsx |
Replaced Field with TextInput for password inputs. Updated error handling logic and added console logging for errors. |
Client/src/Pages/Infrastructure/CreateMonitor/CustomThreshold/index.jsx |
Replaced Field with TextInput . Updated error handling logic and removed hideErrorText prop. |
Client/src/Pages/Infrastructure/CreateMonitor/index.jsx |
Replaced Field with TextInput for various inputs. Updated error handling and refined notification handling logic. |
Check
component's error handling and the introduction of the noHighlightText
prop may relate to the refactoring of the Field
component, which also involves error handling and component structure improvements.Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This PR integrates
TextInput
into the register pageTextInput
to exposename
fieldTextInput
error helper textCheck
component: attn @marcelluscaio please review this for accuracy, resovles following: