Skip to content

Conversation

@vdvman1
Copy link
Collaborator

@vdvman1 vdvman1 commented Mar 11, 2025

Using Turso will allow for created branched databases for each pull request, and local development should be easier since it just uses sqlite.

The branched databases will be named based on the branch name and id/number for the pull request, this will make it possible for the site to automatically use the branched database just using existing environment variables, instead of needing to build on GH:A in order to set new environment variables before deployment.

The branched database will be automatically deleted when the PR is merged or closed, in order to prevent build-up of databases filling up our Turso limits. Unfortunately this does mean that we won't be able to go back to old preview deployments without manually creating the branched database again. This is a suitable compromise though, since for old previews the main database may no longer be compatible with the migrations that would be performed, so the DB would probably need to be created manually anyway in that case

@vercel
Copy link

vercel bot commented Mar 11, 2025

@vdvman1 is attempting to deploy a commit to the vdvman1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Mar 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
mcc-tools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 11, 2025 0:33am

@vdvman1 vdvman1 changed the title feat(internal): ✨ Add notes on the advanced about page for what particular environment variables are for Switch to Turso for the backend database Mar 11, 2025
…ions to be able to access secrets for DB branching
@vdvman1 vdvman1 marked this pull request as ready for review April 30, 2025 04:57
@vdvman1 vdvman1 merged commit 39f4144 into MinecraftCommands:main Apr 30, 2025
1 check failed
@vdvman1 vdvman1 deleted the turso-db branch April 30, 2025 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant