feat: create save, login flow #62
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Marked this as draft because it's non-functional on mobile. If it were ugly, I would've let it slide. |
MinhxNguyen7
requested review from
MinhxNguyen7
and removed request for
seancfong
MinhxNguyen7
requested review from
seancfong
and removed request for
MinhxNguyen7
seancfong
reviewed
Mar 12, 2024
seancfong
reviewed
Mar 12, 2024
seancfong
reviewed
Mar 12, 2024
seancfong
reviewed
Mar 12, 2024
seancfong
reviewed
Mar 12, 2024
seancfong
reviewed
Mar 12, 2024
seancfong
approved these changes
Mar 15, 2024
There was a problem hiding this comment.
@KevinWu098 The modal front-end looks good to me. Not sure if you want this merged into main and merge main into your other branch or directly merge this branch into your lucia/drizzle one.
KevinWu098
added a commit
that referenced
this pull request
Mar 27, 2024
4 tasks
MinhxNguyen7
pushed a commit
that referenced
this pull request
Mar 31, 2024
* feat: ✨ initial tabs setup * style: 🎨 adjust tabs * style: 🎨 adjust bg and block colors * style: 🎨 borders * style: 🎨 set days to 5, simplify styles * style: 🎨 responsiveness * feat: ✨ add columns prop, based on viewport width * style: 🎨 use sean's tabs * chore: 🔧 remove old availability pages * chore: 🔧 rename v2 to default * fix: 🐛 change sample availability date constructor * feat: ✨ setup stores, buttons for saving * fix: 🐛 revert auth changes for easier development * feat: basic modal component * feat: ✨ various modal improvements (esc, backdrop, close icon), styling * feat: get login result * chore: remove unused code * fix: set unsaved state to false only on success * fix: add support for two forms (login, guest) * style: prevent ui shift at smaller screens * chore: create placeholder guest schema * fix: handle mobile/desktop with window width * style: mobile styling of modal * style: decrease button padding on mobile * rename LoginFlow to LoginModal * chore: remove testing code, unused imports * fix: correct guest input binding * feat: ✨ correct login, register to match prisma schema * style: 🎨 make elements more visible w/ bg color * feat: ✨ correct schema and ui to just have username, email, pw * feat: ✨ rename username to displayName * feat: ✨ setup database * chore: 🔧 add dev db to gitignore * refactor: rename boolean stores * refactor: rename to auth-modal * refactor: ♻️ add interface, rename for login data * fix: 🐛 update username to displayName in drizzle schema * chore: 🔧 install dependencies, setup drizzle studio * feat: ✨ update schema * chore: 🔧 uninstall unused lucia dep * feat: ✨ update lucia to v3 * feat: ✨ migrate login/registration * fix: 🐛 update load function for load * fix: 🐛 clear superform warning * fix: 🐛 disable google auth config * feat: ✨ update logout * fix: 🐛 call dotenv config * chore: 🔧 remove testing code * fix: 🐛 adjust lucia config * fix: 🐛 correct postgres client * fix: 🐛 correct schema to use text, not uuid type * chore: 🔧 remove console logs * chore: 🔧 remove log files, update config * feat: ✨ migrate google oauth * chore: 🔧 remove console log * chore: 🔧 update deps * chore: 🔧 update deps pt2 * feat: ✨ disable google auth, correct schema * fix: 🐛 disable google in lucia config * fix: 🐛 correct bugs and errors from #62 * chore: 🔧 extract existingUser function * chore: 🔧 remove console log * fix: 🐛 use scrypt over oslo * chore: 🔧 remove prisma * feat: ✨ update ProfileCard w/ auth buttons and info * feat: ✨ update login ui * fix: 🐛 cleanup login * feat: ✨ update register ui * chore: 🔧 add error into form error message --------- Co-authored-by: Sean Fong <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR is entirely frontend sided as the backend endpoints / db connections haven't been setup yet.
Progress
Screenshots
Future Followup
Note
Currently, this PR is using the old auth for easier development. Those changes should be reverted before merge.
Future Followup
Closes #59