Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: replace formik with react hook form #24

Merged
merged 7 commits into from
May 28, 2023

Conversation

Saba-Var
Copy link
Owner

No description provided.

@what-the-diff
Copy link

what-the-diff bot commented May 28, 2023

PR Summary

  • Removed RememberMe component
    Unnecessary component removed to simplify the code.
  • Modified code editor settings
    Settings for automatic code fixing on save with eslint.
  • Updated LogInForm and NewPasswordForm
    Switched from formik to react-hook-form, removing obsolete code.
  • Removed ResetPasswordRequest component
    Reset password request now managed by backend, making frontend components unnecessary.
  • Renamed and relocated form components
    ResetPasswordRequestForm renamed to SignUpForm and useResetPasswordRequest renamed to useSignUpForm, moving to the sign-up folder.
  • Added CheckboxInputField component
    Created a new shared CheckboxInputField component with tests and types.
  • Updated InputField component
    React hook form integration improved with the addition of a register prop.
  • Added react-hook-form and removed formik
    Switched to react-hook-form, removing the need for the formik dependency.
  • Removed FormikHelpers type
    FormikHelpers type removed from global.d.ts as it's no longer needed.
  • Added rememberMe field
    Added rememberMe field to SignInFormValues for user convenience.

@Saba-Var Saba-Var merged commit db09916 into main May 28, 2023
1 check passed
@Saba-Var Saba-Var deleted the feat/replace_formik_with_react_hook_form branch May 28, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant