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

Database migration #68

Merged
merged 11 commits into from
Dec 10, 2024
Merged

Database migration #68

merged 11 commits into from
Dec 10, 2024

Conversation

n11ik
Copy link
Contributor

@n11ik n11ik commented Dec 9, 2024

No description provided.

@n11ik
Copy link
Contributor Author

n11ik commented Dec 9, 2024

@silvioheinze ich habe die migrations jetzt erstellt und sie funktionieren auch lokal.

Soll ich das Migrieren noch einmal mit Daten testen, before wir die branch meargen?
Für die neu erstellten Tabellen gibt es noch keine Daten, soll ich das trotzdem testen oder testen wir das dann später und machen dann noch eventuell Änderungen.

@silvioheinze
Copy link
Contributor

silvioheinze commented Dec 9, 2024

@n11ik Nimm mal einen der Backups von der production und teste die migration damit auf dem staging. Wenn das klappt, dann können wir mergen.

@n11ik
Copy link
Contributor Author

n11ik commented Dec 9, 2024

Ich habe es jetzt mit dem backup vom 6.12 getestet, noch ein paar Änderungen gemacht und jetzt funktioniert es.

@n11ik n11ik closed this Dec 9, 2024
@n11ik n11ik reopened this Dec 9, 2024
@n11ik n11ik requested a review from silvioheinze December 9, 2024 14:27
@n11ik n11ik merged commit 696600f into main Dec 10, 2024
2 checks passed
@n11ik n11ik deleted the database_migration branch December 10, 2024 07:10
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.

2 participants