chore(deps): clean up dependencies

vevcom / projectNext

Project Next is Omegas new website coming soon

MIT License

6 stars 1 forks source link

chore(deps): clean up dependencies #261

Closed JohanHjelsethStorstad closed 2 months ago

JohanHjelsethStorstad commented 2 months ago

This PR:

Upgrades react-pdf
removes unused typing dependency
upgrades eslint-import-resolver-typescript and eslint-plugin-import

I was not able to upgrade eslint to 9.0.0 as eslint-plugin-import does not support it I think

theodorklauritzen commented 2 months ago

It seems like the upgrade of react-pdf breaks the ombul page

JohanHjelsethStorstad commented 2 months ago

Hah. I tested it in safari and it worked but on chrome it is broken

Paulijuz commented 2 months ago

I think the building issue is a problem with Next js. According to this comment a fix is now ready for the next release. In the mean time setting swcMinify: false in next.config.js seems to fix the problem.

JohanHjelsethStorstad commented 2 months ago

I don't think changing the next Is a great idea. Have used too much time on this already, so I will revert react-pdf I think, and just hope it get better in the near future

JohanHjelsethStorstad commented 2 months ago

Or just wait to merge this

Paulijuz commented 2 months ago

I agree that playing around with the next settings isn't a good idea. We can wait until the next update (which hopefully will fix the issue).

Paulijuz commented 2 months ago

I'll close this pr until then. The other updates were fixed by dependabot anyway