Skip to content

PCR Next.js port #715

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
Open
wants to merge 86 commits into
base: master
Choose a base branch
from
Open

PCR Next.js port #715

wants to merge 86 commits into from

Conversation

@luke-rt
Copy link
Member

@luke-rt luke-rt commented Apr 6, 2025 •
edited
Loading

  • Backend changes: add JWT authentication as an authentication class so isAuthenticated permission class accepts both Platform session id and JWT tokens
  • Update /api/review/autocomplete route to exclude semester in search results

luke-rt and others added 30 commits March 21, 2025 17:04
@luke-rt
Nextjs initialization
793fb80
@luke-rt
Set up tailwind, cva, shadcn
9d1632d
@luke-rt
styling, fonts, began landing page
615d0fa
@rydersitcawich @luke-rt
added about page
a38b6f6
@rydersitcawich @luke-rt
faq page
7e72a72
@luke-rt
styling, scorebox, api helpers
3bc9478
@luke-rt
Nextjs route rewrites
b0e0430
@luke-rt
updated Next.js version, rewrites
1e1e2ce
@luke-rt
Small fixes, began JWT auth migration
b87e529
@luke-rt
JWT tokens working with oidc somewhat working
f95225f
@luke-rt
Removed old auth stuff
a01fc92
@luke-rt
Rollback to tailwind 3 for compatability
ff0621b
@luke-rt
Progress with next-auth configuration
06ad776
@luke-rt
Made some progress with nextauth and middleware
7ed3316
@luke-rt
Trying without nextauth
0c9235a
@luke-rt
JWT token logic working, need to cryptographically verify
14e62ef
@luke-rt
Create test route and setup JWK
7e12b07
@shiva-menta @luke-rt
Add Fetch Locations Management Command
88306fb
@luke-rt
Resolved dev environment bugs
f208676
@luke-rt
Removed duplicate dependencies
e00ecfa
@luke-rt
Got auth middleware working >:)
42cde55
@sritanmotati @luke-rt
rename of token
e95e0b8
@luke-rt
Began working on design
cc35dd7
@luke-rt
Set up debounced course search
025af3f
@luke-rt
Searchbar in header
314f801
@luke-rt
Bump dependencies, upgrade tailwind
7d3d2cb
@luke-rt
Fix expired token bug and cache JWKs
dd3e807
@luke-rt
Use shadcn for searchbar
3d9ae25
@luke-rt
Reconfigure Shadcn, finish searchbar styling
1bd9fea
@luke-rt
Nits
4761cf6
@luke-rt luke-rt requested a review from esinx September 1, 2025 03:25
jamesdoh0109
jamesdoh0109 approved these changes Sep 28, 2025
View reviewed changes
Copy link
Contributor

@jamesdoh0109 jamesdoh0109 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😈

@jamesdoh0109
Copy link
Contributor

jamesdoh0109 commented Sep 28, 2025

Please wait for all downstream PRs to be approved and merged to this base before final merge to master

jamesdoh0109 and others added 5 commits September 28, 2025 17:04
@jamesdoh0109
install cryptography per uv migration
49a65f7
@jamesdoh0109
remove print statements
b17e5cd
@jamesdoh0109
Merge pull request #732 from pennlabs/james/install-cryptography-foll…
4496c46 
…owing-uv-migration

James/install cryptography following uv migration
@luke-rt
Fix test cases (#733)
f5573c2 
Fix test cases to match new autocomplete route. Fix new JWTAuthentication strategy to match spec
@luke-rt
Rename to review-2.0
e9d3ffb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@el-agua el-agua el-agua left review comments

Copilot code review Copilot Copilot left review comments

@jamesdoh0109 jamesdoh0109 jamesdoh0109 approved these changes

@esinx esinx Awaiting requested review from esinx

Assignees

No one assigned

Labels

do not merge

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@luke-rt @jamesdoh0109 @el-agua @rydersitcawich @shiva-menta @sritanmotati