-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update javascript dependencies #326
Conversation
My plan is to update all dependencies to the latest MAJOR version. That will surely break things. |
@knoxfighter Yeah, I wanted that as well. I tried to look into the MapGenerator again and noticed we could need an update of the dependencies. |
Ah, i see, could have created is as DraftPR :) |
It seems to be related to tailwindcss. I'll have a look at it tomorrow or so. |
@knoxfighter you can now have a look at it. |
@knoxfighter is this ready to be pushed to the develop branch? |
Writing it down now, since i currently have a look at it and will fix this the following weeks: The update of react-router from v5 to v6 changed the fundamentals completely, therefore i have to rewrite everything that does something with routes (which is most of it ...) |
Wasn't that bad, got a working UI again now 🥳 |
If anybody wants to have another look, i would love the feedback. |
I can try it out but not before the weekend :) But we can always test in production ;) |
@Mattie112 I merge this PR now, i would love if you test the changes on the |
Did not notice any issues 👍 |
This will update all the javascript dependencies.
I've updated
tailwindcss
to the latest version and moved from the deprecatednode-sass
tosass
.No errors or warnings when building the frontend.