Closed nightzirch closed 3 years ago
This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.
🔍 Inspect: https://vercel.com/nightzirch/secret-toymaker-web/fgkoa0a33
✅ Preview: https://secret-toymaker-web-git-feature-nextjs.nightzirch.vercel.app
The code is still far from optimized considering I want to phase out Reactn in favor of contexts and SWR, but that will require a bigger backend rewrite, so I will leave it for now.
This PR is also not without errors, but those will be fixed in other PRs and tasks. This PR was primarily to rewrite to NextJS, and that has been done. It will build and deploy to Vercel, and that's pretty cool.
The reason for moving over to Vercel's NextJS is primarily due to performance, SEO and simplicity. With NextJS's most recent v10 release, the support for internationalization, server-rendered images and more, the project have lots of opportunities to grow in the future.
The move isn't without complications though. We will need to do a bunch of rewriting and phasing out of the current global state library ReactN in favor of Vercel's own SWR.Turns out we can make ReactN work with NextJS, so we'll keep that in as to not scope this release too ambitiously.To do:
Move some API calls to server-sideMoving this to a task of its own.