search

sledilnik / zdravniki

Website for showing doctors' availability in Slovenia - https://zdravniki.sledilnik.org/
https://zdravniki.sledilnik.org
8 stars 8 forks source link

chore: ⬆️ all deps, but eslint-plugin-react & react-leaflet #468

Closed jalezi closed 6 months ago

jalezi commented 6 months ago

We can not upgrade 2 deps:

As for react-leaflet, there is a workaround, I am pretty sure that eslint-plugin-react can not be upgraded. Most likely more and more dependencies will break the app.

Since create-react-app is not maintained anymore (last update was 2 years ago), the React team recommends using production-grade React frameworks like NextJS, Remix, Gatsby, or Expo for native apps. While frameworks are the preferred choice, the React team also recommends using Vite or Parcel for custom-build processes.

I guess the easiest solution is to use Vite but I have never done it nor used it. Using a production-grade framework needs a major rewrite.

Here is an article on how to migrate to Vite.

jalezi commented 6 months ago

@stefanb on creating PR the playwright test failed. After adding a deploy-preview label` I re-run and it worked.

471 playwright job succeed on first try without 🏷️

https://github.com/sledilnik/zdravniki/actions/runs/8168384693/job/22330362751?pr=471