Remove Next.js - Githubissues

commaai / flash

https://flash.comma.ai

5 stars 14 forks source link

Closed incognitojam closed 3 months ago

incognitojam commented 3 months ago

I'm not sure what I had in mind when I created the app, but we don't need 95% of the features Next.js provides.

remove Next.js
use vite to build
import fonts using fontsource (instead of next/font)
setup eslint (previously using next config)
replace next-plausible with script import in index.html (and removed event tracking in fastboot.js)

This might be an easier first step than switching to a different framework - the react stuff still works!

adeebshihadeh commented 3 months ago

So moving off React here is mostly a testing ground for connect, but I agree this is a good start!

wgenh commented 3 months ago

Should PR #43 be closed?