S24/daniel/create password page

[MinIcecream]

Notion ticket link

Set New Password Page

Implementation description

• Made the CreatePasswordPage which displays the user's email and two password fields, with validation.
• If viewed on a phone in landscape, it becomes scrollable
• Created responsive components for common components.
• Changing password in create password page changes your password. Next time you log in, the updated password is used.

Steps to test

1. Go to localhost:3000/create-password
2. Password fields should throw errors if they don't match or if they're less than 8 characters. Otherwise, it opens the confirmation modal.

What should reviewers focus on?

• Should the route be a PrivateRoute with allowedRoles={AuthConstants.ALL_ROLES}?
• Should I add any other validators? Also, is there a better way I should implement form validation?
• Any coding style/visual things that look off
• Is there a better way to apply different styles based on device orientation?

Checklist

• [X ] My PR name is descriptive and in imperative tense
• [ X] My commit messages are descriptive and in imperative tense. My commits are atomic and trivial commits are squashed or fixup'd into non-trivial commits
• [ X] I have run the appropriate linter(s)
• [ X] I have requested a review from the PL, as well as other devs who have background knowledge on this PR or who will be building on top of this PR

Mobile:

Tablet:

[jerry-cheng5]

Great work! love the different components you made to make the code reusable.

Just a few issues that I found.

[trinity-y]

Should the route be a PrivateRoute with allowedRoles={AuthConstants.ALL_ROLES}?

i did this in my branch by the way so no need to change this 🙏

[JustinScitech]

These changes look good, nice work.

The logo for the humane society does look a little blurry though, in the future we could probably ask the design team to send an SVG to minimize image compression on the page. Would be useful for the rest of the pages as well.

[jerry-cheng5]

code looks good to me!

if you tested the functionality, it's good to merge

[jerry-cheng5]

Oh there's some linter errors, can you run the linter script?

[laks0407]

Looks good to me. nice work!