Add password requirements to login page

[jaggederest]

closes #90

Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

[changeset-bot]

⚠️ No Changeset found

Latest commit: 3c22f8d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[wrn14897]

closes #90

Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

Thanks for fixing the issue. Do you mind sharing the screenshot ?

[jaggederest]

closes #90
Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

Thanks for fixing the issue. Do you mind sharing the screenshot ?

[wrn14897]

closes #90
Would appreciate some feedback on placement and styling - wanted to fit it into the Label element but thought it might be better after the form field

Thanks for fixing the issue. Do you mind sharing the screenshot ?

I think it would be cleaner if we have a list of policies underneath. Ideally policy checks automatically. But for now I think we can just leave it as a normal list.

[jaggederest]

Nice, will do. We'll see how I get along with the checking-as-satisfied section, but at the very least a static list.

[jaggederest]

Kind of a funky SVG but does the thing:

Not sure why the useEffect call is getting {password: 'something'} instead of a raw string.

[jaggederest]

New look with bootstrap SVGs:

[MikeShi42]

final few nits I'd love to see if we can squeeze in - but things look great otherwise :) like the nice touch of the confirm password label being dynamic as well

[jaggederest]

Local version is failing with

./src/useConfirm.tsx:2:0
Module not found: Can't resolve 'jotai'
  1 | import * as React from 'react';
> 2 | import { atom, useAtomValue, useSetAtom } from 'jotai';
  3 | import Modal from 'react-bootstrap/Modal';
  4 | import Button from 'react-bootstrap/Button';
  5 | 

Import trace for requested module:
./pages/_app.tsx

https://nextjs.org/docs/messages/module-not-found

Which is a weird one because it's installed and worked previously 🤷

Anyway, no screenshot from this round for that reason but the changes should be in place

[MikeShi42]

looks great!

[MikeShi42]

@jaggederest you might need to run your setup with --build to rebuild the image to include the new packages/setup. Lmk if that doesn't work for you though