indico / indico-checkin-pwa

MIT License
2 stars 5 forks source link

Replace CRA with Vite #69

Closed AjobK closed 2 months ago

AjobK commented 2 months ago

Problem

The compilation process for CRA is long and official support is also being cut off.

Solution

Moving to Vite, which is significantly faster!

tomasr8 commented 2 months ago

Tried to do a test deploy and the build fails for some reason: https://paas.cern.ch/k8s/ns/indico-checkin-test/builds/indico-checkin-pwa-2-17/logs Will have to dig into why

AjobK commented 2 months ago

Not sure why but the PR seems to close every time I rebase (while testing the new service worker)

github-actions[bot] commented 2 months ago

📝 Lighthouse report

github-actions[bot] commented 2 months ago

📝 Lighthouse report