update dependencies and switch to nextjs links. Moves page into _app!!!!#33
update dependencies and switch to nextjs links. Moves page into _app!!!!#33Nexite wants to merge 11 commits intocodeday:masterfrom
Conversation
slug and title now in getstatic or getserverside props!!!!
oohwooh
left a comment
oohwooh
left a comment
There was a problem hiding this comment.
looks good at initial glance. We still need to test CRUD form functionality before merge, and I am currently too tired to trust myself to test that adequately.
|
useTopo is a function you wrap your nextjs config with and it uses webpack to replace the normal topo links (including with as="a" for buttons and boxes) with nextjs compatible ones. this means that you do not have to do anything special to get the links to work, it just happens automagically! |
based on my testing everything seems like it works fine, I'm not super familiar with all of the forms though. |
2 participants
title and slug properties now set in getstaticprops or getserverside props. prevents re-rendering the component every time (this stops duplicate requests for contributors in footer and increases performance).
Also moved moment-timezone out of _app, reduced first load js by about 50kB.
Switching to nextjs compatible links should make pages load faster and makes user experience much smoother!