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.

Sign up for GitHub

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

Jump to bottom

Push rearchitecture to prod #24

Merged
merged 108 commits into from
Aug 13, 2023
Merged

Push rearchitecture to prod #24

merged 108 commits into from
Aug 13, 2023

Conversation

edwardshturman
Copy link
Member

See #10

@edwardshturman
Fix sidebar layout shift
bd6ecb7 
Conditionally rerender only Invite button rather than the whole sidebar
@edwardshturman
Update Next to latest v13
ec0cc28
@edwardshturman
Extract SessionProvider to client component
7053d1a
@edwardshturman
Add App Router root layout
a56f573 
Notable differences:
• Metadata API
• Extracted Auth.js `SessionProvider` to new `AuthWrapper` client component
• Extracted Sidebar conditional rendering to component-level

Known issues:
• It's bugging out about the styles
• There was something with the `children` not being a valid React child or something like that
@edwardshturman
Extract conditional Sidebar rendering to component
d02c8cf
@edwardshturman
Use new usePathname client component hook
77718e3
@edwardshturman
Extract sign in button to client component
7fce098
@edwardshturman
Migrate sign in page to App Router
c784115 
Notable differences:
• Extracted sign in button to its own client component
• Removed `next/head`, now taken care of by Metadata API
• Returning a redirect object in `getServerSideProps` → `next/navigation` redirect

Known issues:
• Styles not working (todo switch from styled JSX)
• Sidebar rendering "Loading..."
@edwardshturman
Consolidate user APIs
16884d7
@edwardshturman
Migrate Auth.js APIs to App Router
2b195fa
@edwardshturman
Migrate cue generation API to App Router
0ebef49
@edwardshturman
Migrate card invites to App Router
10c1c3a
@edwardshturman
Optimize sidebar code
e3c61d8
@edwardshturman
Fix Edge Function config export
d813436
@edwardshturman
Fix sidebar layout shift (#9)
b732a7e 
Conditionally rerender only Invite button rather than the whole sidebar
@edwardshturman
Keep dev up-to-date with main
621f10a
@edwardshturman
Add promo source for invites
5a5f9f8
@edwardshturman
Add auth policies & logic (#11)
5ab2c18
@edwardshturman
Move /profile route to App Router
26eb9a8
@edwardshturman
Move user fetching and redirecting into async page
a13f3c8
@edwardshturman
Add session callback to pass Google ID
5e44f67
@edwardshturman
Remove /api/user route handlers
f274f41 
Will need to update dependents to call new server-side user management functions
@edwardshturman
Expand user management functions
b15e89e
@edwardshturman
Distinguish fetching user session vs. db data
cd41587
@edwardshturman
Fix conditional for checking if a user exists
2ffd1eb
@edwardshturman
Create user if they don't exist on sign-in
a53edd9
@edwardshturman
Add invite validation logic
8fc2d96
@edwardshturman
Dynamically set a user's invitesRemaining
12a2003
@edwardshturman
Add invite-only auth policy logic
fc1abce
@edwardshturman
Rename InviteCode model to Invite for clarity
ff8ed0a
@edwardshturman
Keep consistent style naming across Routers
4a1057f
@edwardshturman
Add generation construction helper function
4d8285a
@edwardshturman
Ensure live generations data is fetched, check age
951f484
@edwardshturman
Push to user generations on generation call
6192a11
@edwardshturman
Limit generations to 10 a week
160f91e
@edwardshturman
Handle user hitting rate limit gracefully
b9b7036
@edwardshturman
Tokenize notes and generated cues to get cost
496c863 
Early experiments for #21
@edwardshturman
Remove unused icons
a505840
@edwardshturman
Migrate invite cards to Vercel KV
f0cdbe1
@edwardshturman
Move invite code page to App Router
d48fa48
@edwardshturman
Clear invite code search param on /profile
666dac5
@edwardshturman
Remove abstraction of invite status
2fd353e 
Instead, just return the invite info and logic handled as needed by caller
@edwardshturman
Write helper function for getting all invite codes
e0260a8
@edwardshturman
Make invite-only auth policy more restrictive
275fc64 
This lines up with reality: the amount of students that can have invites < the amount of USF students
@edwardshturman
Move /redeem to App Router
a14475b
@edwardshturman
Prevent signed-in users from using invite codes
691abc2
@edwardshturman
Approve auth if user exists (lol)
3105bef
@edwardshturman
Remove Pages Router
b103813
@edwardshturman
Fix access denial for existing users (again lol)
fa0d8c2
@edwardshturman
Fix lint issues
80f03e9
@edwardshturman
Bump to version 0.5.0
2b71551
@edwardshturman
Merge branch 'dev' into meta/next-app-router-migration
3da078a
@edwardshturman
Rearchitecture (#10)
2528bd1 
See PR for more info
@edwardshturman
Fix scripts using Doppler
35be6b2
@edwardshturman
Fix NextRouter not mounting
d6ab3c6
@edwardshturman
Test rearchitecture on staging (#23)
1996a94 
See #10
@vercel
Copy link

vercel bot commented Aug 13, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cue ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 13, 2023 0:32am

@edwardshturman edwardshturman merged commit e1d9274 into main Aug 13, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@edwardshturman