Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

Upgrading to Next.js 13 #780

Open
1 of 5 tasks
spencerwooo opened this issue Oct 28, 2022 · 3 comments
Open
1 of 5 tasks

Upgrading to Next.js 13 #780

spencerwooo opened this issue Oct 28, 2022 · 3 comments
Assignees
Labels
feature request New feature or request

Comments

@spencerwooo
Copy link
Owner

spencerwooo commented Oct 28, 2022

  • Upgrade deps to Next.js 13
  • Adapt new app directory and layout
  • Rewrite of data rendering (also data fetching with React server components)
  • Rewrite of Loading and Error layout
  • ...
@spencerwooo spencerwooo added the feature request New feature or request label Oct 28, 2022
@spencerwooo spencerwooo self-assigned this Oct 28, 2022
@aidenlx
Copy link
Contributor

aidenlx commented Feb 19, 2023

Using app directory and layout could help improving page loading significantly, since it can allow dynamicly load large libs like plyr, pdfjs, etc based on specific path. Currently we have to load everything since the website can't know the content for certain path ahead of time.

Would you be interested in accepting PR regarding app directory refactoring, or collabrate on such refactoring in the future? I think the fetch API refactoring on API route is the first step towards this goal, since the new data fetching system for server components is built on top of the native fetch() Web API.

@aidenlx
Copy link
Contributor

aidenlx commented Feb 20, 2023

one potential objective: app/route in favor of pages/api (available in nextjs 13.1.7, currently in canary)

Note
route handler implement an isomorphic API to support both Edge and Node.js runtimes so functions like ResponseCompat and NodeRequsetToWeb is no longer required.

@spencerwooo
Copy link
Owner Author

Would you be interested in accepting PR regarding app directory refactoring, or collaborate on such refactoring in the future?

@aidenlx Certainly, but I don't think I can spare too much time on this project in the foreseeable future. If you are interested, go for it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature request New feature or request
Projects
Development

No branches or pull requests

2 participants