-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat: SSR POC #1536
base: master
Are you sure you want to change the base?
feat: SSR POC #1536
Conversation
Thanks for your pull request! The preview of your changes is available at https://ssr.preview.coursetable.com. |
Size Change: 0 B Total Size: 3.03 MB ℹ️ View Unchanged
|
I haven't wired it to any deployment yet. To test the build output, run |
@Josh-Cena is this still viable/ something we want to do |
I definitely want to do this eventually, but not sure if it yields any immediate value. We have to migrate off Vercel static serving first. |
507a9e0
to
28c12bf
Compare
ef76603
to
3f2e6c5
Compare
This is needed for #1427 to work.
@bearsyankees My website is fully server-rendered with react-helmet. You can check the impl here:
The code here is slightly adapted to make bun work.
Next we need to figure out how to serve these pre-rendered HTML so they get picked up by social media.