-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update the template #2
base: main
Are you sure you want to change the base?
Conversation
|
||
```sh | ||
$ npx wrangler@latest d1 migrations apply cdph-starter |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing latest
since npx
will use the version listed in the package.json
anyways.
|
||
```sh | ||
$ npx wrangler@latest d1 create starter # or name it something of your choice | ||
$ cp wrangler.toml.example wrangler.toml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added this missing step, setting up the wrangler.toml
.
|
||
pages_build_output_dir = "./dist" | ||
|
||
compatibility_flags = ["nodejs_compat"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The worker wouldn't build for me without this flag.
Hey, I've just used the template to deploy an app and I've had to do a couple of changes to make it work for me.
I've updated the README and the
wrangler.toml.example
file to match my experience.I've also updated the
package.json
to match.I've removed the
migrations
dir file from.gitignore
because it makes sense to include migration files by default so the project can be worked on by multiple people.