-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate to Next.js #25
Comments
Next.js relies on react and react-dom packages in Node scripts. It's possible to mock them and point to necessary files, however it seems like they are run in |
@ezhlobo hey, do you need help with it? |
@nixololo my main issue is that I don't know how to hide Ideally I don't want to expose what is used under the hood and provide own framework, so it's important. |
I attached working branch. I successfully ran nextjs behind
I want to get a router that can be transformed into something that gives the possibility to implement steps mentioned above. Also it seems like we need to invent new commands. I'm thinking about something like this:
PS. Even though my main focus is to implement local development and dynamic app, I need to be careful and pay attention on all the constraints we have from SSR and Static app. |
@ezhlobo when you're talking about "Dynamic app" do you mean implementing of client side routing? Like we receive an html with one script tag, and then receive a bundle of the whole app. Then we use routing to navigate to other routes directly in a browser |
@viscoze yes, "Dynamic apps" are those that understand what to output in runtime, not on server or via pre-building all pages. |
There are two key advantages for us that Next.js has over CRA:
Due to them I think it makes sense to run an experiment and see if we can build our framework on top of Next.js that covers our needs.
Progress
tree/migrate-to-nextjs
dreact app
start
(plusbuild
)dreact env-sync
dreact test
dreact lint
js
styles
dreact book
start
(plusbuild
)Useful new features
.gitignore
contains necessary filesThe text was updated successfully, but these errors were encountered: