(feat) Add the ability to change password from edit profile form by coderatomy · Pull Request #968 · unstructuredstudio/zubhub

coderatomy · 2023-10-26T12:41:31Z

Summary

fixes #904
Adds ability to change password from /edit-profile

Changes

Improves validation and error handling in the Edit Profile form
Improves how form submission is handled. Utilizing the handleSubmit function from formik such that form submission is immediately cancelled if any validation errors exist. Sometimes our custom validation can have some loop holes.

Screencasts

Screencast.from.2023-10-26.15-27-09.webm

coderatomy · 2023-10-26T12:48:44Z

+      "changePassword": {
+        "newPassword": "Change Password",
+        "confirmNewPassword": "Retype new password",
+        "error": "<1/> {{errorMessage}}"


add a custom Component and an error message

coderatomy · 2023-10-26T12:49:42Z

                name="signup"
                noValidate="noValidate"
-                onSubmit={e => editProfile(e, props, toast)}
+                onSubmit={props.handleSubmit}


utilizing the formik handleSubmit for better security in terms of validation

coderatomy · 2023-10-26T12:50:39Z

    }),
    validationSchema,
+    handleSubmit: (values, { props }) => {
+      return editProfile(values, props, toast)


assigning the editProfile with the validated data

coderatomy · 2023-10-26T12:51:36Z

+      is: (newPassword) => newPassword && newPassword.length > 0,
+      then: Yup.string().oneOf([Yup.ref('newPassword')], 'Passwords must match').required('Retype new password'),
+      otherwise: Yup.string().notRequired(),
+    }),


improving the schema for better error handling

coderatomy · 2023-10-26T12:54:16Z

+                            (props.errors['password'] &&
+                              <Trans t={t} i18nextKey="editProfile.inputs.changePassword.error">
+                                <ErrorIcon fontSize="small" /> {{errorMessage: props.errors['password']}}
+                              </Trans>


using the Trans component from react-i18next to improve the field error message with Icons

NdibeRaymond · 2023-10-29T21:50:01Z

hello @coderatomy , you did a great job here! Thanks for that! few things that I'd like us to deliberate on:

have something against having the option to change the password alongside other things here (I didn't follow the conversation on the issue or I would have said the same thing there. I am also going to mention @srish and @tuxology here so they can give their opinion). It's been the standard practice to separate these two flows as one deals with a general change of data while the other deals with a kind of change that can affect a user's ability to log in. We can have a link from the edit-profile page that takes us to the change-password page, but both should not be on the same page. additionally, there should be some form of re-authentication before a user can be redirected to the change-password page (I think we missed implementing this before but it is important).

What I propose is something like this [some link on the edit-profile page] ------------> [page where the user re-authenticates with their old password] -------------------------------> [change-password page]

tuxology · 2023-10-30T18:26:42Z

@NdibeRaymond I agree about this as well. I see that @srish approved this more simple design here (#904 (comment)). I think the rationale why we have mixed both "Edit Profile" and "Account Settings" workflow is because the Edit Profile page allows you to change username, location, personal details etc. Which is very similar to "account" related info. Hence, we can, for now also allow changing password via that workflow as well. We can have a dedicated change password/account settings button some where which can do a similar workflow as yours and then we can phase out this change. We would need UX input for that and I think it would look similar to your approach

chaitanya-cloudknox · 2023-10-30T21:52:37Z

    color: 'var(--primary-color2)',
  },
  fieldHelperTextStyle: {
+    '&.MuiFormHelperText-root.Mui-error': {


Try sending the classnames to mui component rather than using their generated class names. it would hassle when upgrading libraries as they change how the classnames (or styles) (they usually support it) are created or update dom structure

<ErrorComponent classNames={{
errorComponent: classNames.myErrorClassName
}} />

chaitanya-cloudknox · 2023-10-30T22:01:20Z

-                      <InputLabel
-                        className={classes.customLabelStyle}
-                        htmlFor="password"
+                  <Grid container style={{ display: 'block'}}>            


Why are you changing the grid system to block? its just gonno affect responsiveness

FYI If you aare changing to block system just use div instead

chaitanya-cloudknox · 2023-10-30T22:03:17Z

+                          endAdornment={
+                            <InputAdornment position="end">
+                              <IconButton
+                                aria-label="toggle password visibility"


labels need translations

chaitanya-cloudknox · 2023-10-30T22:04:27Z

  username: Yup.string().required('required'),
  user_location: Yup.string().min(1, 'min').required('required'),
-  password: Yup.string().required('required'),
+  password: Yup.string().required('Your password is required'),


the errors needs translation

aqsaaqeel · 2023-11-14T19:30:02Z

hey @coderatomy please link this PR to #984 so that redundant work can be avoided. Once this PR is merged I can work on moving the edit profile to the sidenav. Thanks!

coderatomy · 2023-11-15T15:58:01Z

hey @aqsaaqeel , i think you can go on with this, just go through the comments here to get the context, then get out what might be useful from this PR and use it. My main priority now is on the Migrations. Kindly help me with that.

aqsaaqeel · 2023-11-15T19:48:16Z

hey @aqsaaqeel , i think you can go on with this, just go through the comments here to get the context, then get out what might be useful from this PR and use it. My main priority now is on the Migrations. Kindly help me with that.

Alright!

coderatomy added 2 commits October 26, 2023 15:29

add change password ability from /edit-profile

d5b7255

remove logs

ff0d432

coderatomy requested review from NdibeRaymond and tuxology as code owners October 26, 2023 12:41

coderatomy commented Oct 26, 2023

View reviewed changes

srish mentioned this pull request Oct 30, 2023

Move change password feature to a new tab #984

Open

chaitanya-cloudknox reviewed Oct 30, 2023

View reviewed changes

aqsaaqeel mentioned this pull request Nov 15, 2023

Feat/984: Add Edit Profile to the sidenav #999

Open

Conversation

coderatomy commented Oct 26, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Screencasts

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

NdibeRaymond commented Oct 29, 2023

Uh oh!

tuxology commented Oct 30, 2023

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

aqsaaqeel commented Nov 14, 2023

Uh oh!

coderatomy commented Nov 15, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

aqsaaqeel commented Nov 15, 2023

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

coderatomy commented Oct 26, 2023 •

edited

Loading

coderatomy commented Nov 15, 2023 •

edited

Loading