-
Notifications
You must be signed in to change notification settings - Fork 1
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
Migrationer #8
Migrationer #8
Conversation
@malmz @The1Penguin sanity check please 🥺 |
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.
LGTM
Co-authored-by: Simon Renhult <[email protected]>
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.
Ser bra ut, känns som något som skulle kunna lösas med ett migrations verktyg men några av våra tjänster kan väl vara lite wild and crazy.
Lite sus transaktionslösa operationer som förmodligen är fine för sqlite har typ inte concurrency och det kör innan appen startas så w/e.
beep boop, get merged bozo |
TV mannen skulle gärna behövas pillas lite på, saker som hade varit nice är typ
Dessa skulle garanterat kräva ändringar i vårt databas schema, så vi bör ha något system för migrations.
PR:et inkluderar ett simpelt system för migrationer och en migration som ändrar så att våra strängar inte längre har maxländger (onödigt, och trivialt att migrera p.g.a hur sqlites typer funkar).
Skapar detta PR för att få lite code-review sanity check innan jag mergar till main :)