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

Migrate to Entity Framework Core #184

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

kvn-stgl
Copy link
Collaborator

This PR migrates the entity framework to the entity framework core. It also supports SQLite as database option now.

This PR migrates from the entity framework to entity framework core. It also supports SQLite as database option now.
@Flozi95
Copy link
Contributor

Flozi95 commented Jan 2, 2019

Sicher, dass bei SQLite die Migrationen zuverlässig funktionieren? Hatte da mal etwas Probleme.

@kvn-stgl
Copy link
Collaborator Author

kvn-stgl commented Jan 3, 2019

Ich konnte bis jetzt kein Problem damit feststellen. Nur mit dem EF6 gab es mehrere Probleme, weswegen ich auf das EF Core gewechselt bin. Falls du aber auf andere Probleme gekommen bist, lass es mich gerne wissen :)

@Flozi95
Copy link
Contributor

Flozi95 commented Jan 3, 2019

s. https://docs.microsoft.com/de-de/ef/core/providers/sqlite/limitations

es gibt ein paar EInschränkungen. Ob wir mit denen leben können?

@kvn-stgl
Copy link
Collaborator Author

kvn-stgl commented Jan 6, 2019

Ich sehe dort nichts wirklisch dramatisches. Wir haben intern keine wirkliche Logik in der Datenbank, sondern nutzen diese eigentlich nur zum speichern der Daten. Das einzige ist, dass wir keine Spalten mehr löschen oder den Namen dieser ändern können .. Was wir aber auch nicht brauchen, denke ich.

@KirschbaumP
Copy link
Contributor

Die Frage ist doch brauchen wir überhaupt eine Datenbank?
Wir haben so geringe Datenmengen das man die Daten auch in mehreren Dateien speichern könnte(z.B. config.json, operations.json,...). Jede Datei kann mit einem Schema versehen werden, ...

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.

3 participants