search

timwis / jkan

A lightweight, backend-free open data portal, powered by Jekyll
https://jkan.io
MIT License
219 stars 309 forks source link

Force upgrade NPM everything #196

Closed BryanQuigley closed 1 year ago

BryanQuigley commented 1 year ago

We will be able to remove many components when removing the frontend editing components, but without these changes I couldn't get webpack to build - which means properly testing it was hard.

timwis commented 1 year ago

Happy to test this locally—just to flag, though, that #203 is close enough that I'll be able to remove the built-in admin/editing UI soon, which will remove like 90% of the JavaScript. Not that there are many dependencies, to be fair.

I'm at a wedding at the moment, so it might be a few days before I can review—from the looks of it, it's just the build tools, so I'd say if it builds successfully for you, feel free to merge so you're not blocked.

BryanQuigley commented 1 year ago

It can wait - I had to do this to look at removing any parts of the old UI editing bits. But your mileage might be better.

Enjoy the weeding!