Update project depedencies #7
Conversation
There was a problem hiding this comment.
Hej och stort tack för din PR! Här kommer en ULTRAsen review, så vi förstår om du kanske inte kommer att fixa det 😅
Mycket bra med uppdaterade dependencies, och det är helt rätt att nyare versioner av Gatsby kräver att man importerar, men hade föredragit att importera allt i css-modulerna. Alltså "import * from blabla.module.css", eftersom det är ett undantagsfall att en klass i modulen inte används. Det är även så gatsby rekommenderar i sin dokumentation :) https://www.gatsbyjs.com/docs/how-to/styling/css-modules/
Då behöver man inte heller göra om klassnamnen.
Makanz
force-pushed
the
update-dependencies
branch
from
eef3e14 to
03aa595
Compare
|
Har fixat css importen enligt rekommendation och uppdaterat npm dep till senaste versionerna. Hittade ett problem med gatsby-plugin-gdpr-cookies som inte har stöd för React 18. |
Uppdaterat till senaste versionen av Gatsby och React.
Importen av css-modules har ändrats i senaste versionen av Gatsby. Tror att jag löst det i alla komponenter men det kan behöva ses över.