paretOS icon indicating copy to clipboard operation
paretOS copied to clipboard

`ChangePassword.tsx` react-bootstrap -> react-hook form migration

Open mikhael28 opened this issue 3 years ago • 4 comments

Description

There are several areas of the repo that are still using the Bootstrap form components FormGroup, FormControl, ControlLabel, and HelpBlock. These should be migrated to Material UI (FormControl, FormControlLabel, and FormHelperText). In addition, we want to have those components refactored to Hook's from classes (if they aren't already), and to use the react-hook-form npm library to improve the performance of our forms. Review this old Issue for more details. You can refer to Login.js to see what successful implementation looks like.

If you'd like to help, please respond to this issue!

A comprehensive refresher on Material UI forms: https://smartdevpreneur.com/every-material-ui-form-component-explained-mui-v5/

Official docs: FormControl FormControlLabel FormHelperText

mikhael28 avatar Aug 20 '22 01:08 mikhael28

Hi @mikhael28, could you assign me to the this issue?

AleksandrKurilov avatar Sep 22 '22 02:09 AleksandrKurilov

There you go @AleksandrKurilov let me know if you get stuck.

jeremydthomas avatar Sep 22 '22 03:09 jeremydthomas

@jeremydthomas thank you

AleksandrKurilov avatar Sep 22 '22 12:09 AleksandrKurilov

Thank you @AleksandrKurilov

mikhael28 avatar Sep 26 '22 16:09 mikhael28