Skip to content
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

Fix errors when running the project the first time #1796

Merged
merged 9 commits into from
Mar 24, 2023
Merged

Conversation

Janpot
Copy link
Member

@Janpot Janpot commented Mar 24, 2023

  • Avoid crashing on first start because of missing queries file
  • Initialize project only once when the dev command runs
  • More fine grained initialization where we decide when and how to generate initial project files. Also run it in the application process for now colocated with the other project code. We'll see for the future how we can share this functionality between cli process and next.js process

Closes #1763

@Janpot Janpot added the core Infrastructure work going on behind the scenes label Mar 24, 2023
@Janpot Janpot changed the title Better project init Fix issues on project initialization Mar 24, 2023
@Janpot Janpot marked this pull request as ready for review March 24, 2023 12:15
@Janpot Janpot changed the title Fix issues on project initialization Fix errors when running the project the first time Mar 24, 2023
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Mar 24, 2023
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Mar 24, 2023
@Janpot Janpot merged commit 68752ea into master Mar 24, 2023
@Janpot Janpot deleted the project-init branch March 24, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

First run adding serverside query doesn't work
1 participant