Add support for Prisma to create-keystone-app

[timleslie]

To test this you can run the script locally with:

 node packages/create-keystone-app/bin/cli.js

[request-info]

It looks like we could use some more details. Good titles and descriptions help us fix things faster. If you can provide more information please update this issue.

[changeset-bot]

🦋 Changeset detected

Latest commit: 856b0ab

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
create-keystone-app	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[JedWatson]

@timleslie just picking default values (more or less) without a database created leads to a hard error. Can we improve this?

(related; I was of the impression Prisma migrate can create the database if/when it needs to, assuming the server (i.e localhost, default port and auth) is available?)

Any meaningful fix here is possibly/probably out of scope for this PR, but throwing an error and crashing the process from the default choice seems like a poor experience for first users. We could catch the error, point them at the docs, and offer to continue anyway saying "You'll need to create your database after this" and that would be much nicer.

[JedWatson]

Related to that last comment, when I skip the "test database" step, and follow the instructions it errors out without helping me know what next step to take to resolve the problem (spoiler: follow the docs to create a database)

I think we should do better here; we can suggest a command that will probably do what they want; or at minimum point at the guide.

If I was trying Keystone out for the first time here with less knowledge, the current experience would just be a roadblock I'd walk away from.

[timleslie]

@timleslie just picking default values (more or less) without a database created leads to a hard error. Can we improve this?

I think in this case we should remove the default database string. At the moment there is no valid default database string which will work. We should structure the instructions such that they know that they need this value right up front.