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

Planetscale recipe #3053

Open
Vandivier opened this issue Dec 11, 2021 · 5 comments
Open

Planetscale recipe #3053

Vandivier opened this issue Dec 11, 2021 · 5 comments
Labels
status/ready-to-work-on This issue is up for grabs

Comments

@Vandivier
Copy link
Contributor

Vandivier commented Dec 11, 2021

What do you want and why?

As a developer, I would like to be able to easily integrate planetscale db, blitz, and render.com

Possible implementation(s)

  1. modify blitz install render to support a planetscale-compatible YAML
  2. make blitz install render-planetscaledb or similar; basically a new recipe
  3. no recipe, just update deploy to render docs to include YAML that works with planetscale db.
@Vandivier
Copy link
Contributor Author

on second thought maybe this should be a project template instead of a recipe? rationale: currently when I npx blitz@latest new myApp eventually it will tell me something like:

Success! Next steps:
cd myApp
blitz prisma migrate dev
blitz dev

however I think we shouldn't migrate with planetscale, so in theory users could learn to "just ignore that" but this isn't really ergonomic

@Vandivier
Copy link
Contributor Author

Vandivier commented Dec 11, 2021

tangentially related:
#3058 (comment)
and:
prisma/prisma#7292

@Vandivier
Copy link
Contributor Author

another pain point is that the db.$reset helper seems to fail when connected to planetscale, maybe bc it calls migrate underneath

image

@beerose beerose added the status/ready-to-work-on This issue is up for grabs label Dec 14, 2021
@beerose
Copy link
Contributor

beerose commented Dec 14, 2021

on second thought maybe this should be a project template instead of a recipe? rationale: currently when I npx blitz@latest new myApp eventually it will tell me something like:

Success! Next steps:
cd myApp
blitz prisma migrate dev
blitz dev

however I think we shouldn't migrate with planetscale, so in theory users could learn to "just ignore that" but this isn't really ergonomic

I think the recipe makes sense. If user creates a new Blitz app, then can first play with it with Sqlite, and then install planetscale recipe. This recipe should remove the migrations folder and do all the required changes in the db directory. It can also remove references to $reset if needed or replace it with something that works with planetscale. What do you think?

@Vandivier
Copy link
Contributor Author

Makes sense to me! I guess that's the same pattern we follow for any remote db (postgres, etc) currently anyway

@beerose beerose changed the title render recipe for use with planetscale recipe for use with planetscale Feb 28, 2022
@beerose beerose changed the title recipe for use with planetscale Planetscale recipe Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/ready-to-work-on This issue is up for grabs
Projects
None yet
Development

No branches or pull requests

3 participants