-
Notifications
You must be signed in to change notification settings - Fork 227
Next.js 15 + React 19 + Happenings + dependency upgrades #1516
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.
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
base: main
Are you sure you want to change the base?
Conversation
Turbopack's so fast that we send too many API requests! To fix this, we use an authenticated token, which does work for now.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hey! Great work. This is eligible for the $50 USD for the submission. I don't love how dark the above the fold is, but I love all the upgrades you made to the site and want to say overall - amazing work! I don't think it's enough of a visual change to count as a redesign though. I'd love to see more visual changes. |
Just tested this PR with the Vercel vars and it works :D |
This is a large PR! I'd guess that it'd be easier to review commit-by-commit. Do let me know if there's any way I can help :)
A large part of the changes comes down to codemods (e.g. adapting to
next/link
's new behaviour, updating the code to work with the latestnext/image
, and swapping outreact-reveal
forreact-swift-reveal
)This PR:
LOOPS_TOKEN
on VercelHAPPENINGS_SECRET_KEY
- should be a random stringcdn.glitch.com
URLs to Hack Club's CDN. Glitch is shutting down in just over a month (all web hosting services are being pulled), so this change ensures that images on the site don't break when that happens.It also has the following development benefits to make the site easier (and faster!) to work on:
ENABLE_EXPERIMENTAL_COREPACK=1
on Vercel! I've deployed this to Vercel and it works finereact-reveal
forreact-swift-reveal
. The reason being that unlike react-reveal, react-swift-reveal supports React 19 and has the same API surfaceyarn build
able in developmentI've built the site on Vercel and it works fine. I've also tested most of the pages (shoutout to #mahads-airport for the help :D)