sergiodxa / remix-i18next

The easiest way to translate your Remix apps
https://sergiodxa.github.io/remix-i18next/
MIT License
602 stars 44 forks source link

Support for building with Remix 2.0.0 #151

Closed ahockersten closed 1 year ago

ahockersten commented 1 year ago

No big changes required, it seems. The main one affecting this project is that route.handle is now of type unknown, so I added some additional casts in order to avoid type checking errors.

In addition to this, Remix requires Typescript 5.1.0 or later. This, in turn, requires upgrading @typescript-eslint to at least 5.61.0 (I went with 5.62.0 since that's the last version before 6.0.0).

Of course, just running npm install means we get a bunch of other packages upgraded. I just went with it since none of the packages seem like they would cause breakage on upgrade but of course it's tricky to make guarantees about this.

Obviously I'm not very familiar with the internals of this project (I'm just a happy user :)), so it's perfectly possible this patch is not enough, but it might be an OK start at least.

sergiodxa commented 1 year ago

Thanks a lot of helping with the support for Remix v2!