You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As we are using nextJS 14, the idea is to move from the pages to the App router, which has a better way of loading server data before displaying the pages. e.g. that will improve how we load ENS cache data (Feat under review here #175) as we want to load only once, and _app.tsx doesn't support any of the server data fetch, besides hacky ways that can cause more harm than good.
✔️ Solution
Move all the page routes under pages/ to app/.
Other requirements:
Leverage data-fetching on the server in the layout.tsx. i.e. Avoid unnecessary network roundtrips.
Evaluate the need for ISR generation now that we can access the ENS information cached and on the fly in the server.
📈 Subtasks
Port routes from pages/ to app/ router.
Make adjustments between client and server components (as necessary)
Refactor ENS cache initial load to be a single call with page load (i.e. use layout.tsx)
Evaluate ISR utility and refactor (as necessary)
Fix test suites (as necessary)
🎯 Definition of Done
All the pages render as expected.
ENS initial cache data is available for context in a single trip.
Before making that move, a few changes will be made to make it less painful in a single shot. I will split the bullet into necessary and optional ones related to that issue specifically
📄 Context
As we are using nextJS 14, the idea is to move from the pages to the App router, which has a better way of loading server data before displaying the pages. e.g. that will improve how we load ENS cache data (Feat under review here #175) as we want to load only once, and
_app.tsx
doesn't support any of the server data fetch, besides hacky ways that can cause more harm than good.✔️ Solution
Move all the page routes under
pages/
toapp/
.Other requirements:
📈 Subtasks
🎯 Definition of Done
The text was updated successfully, but these errors were encountered: