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

basic postgresql support #19

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

Conversation

avioletheart
Copy link
Contributor

i don't really expect this to be merged, but if anyone wants to run the server on postgres here's a patch that might work. i did do a little testing (can create a regular/sandbox game, can perform basic moves, can create new timelines) but nothing extensive

@Oliveriver
Copy link
Owner

Ideally I would like to support both providers and switch between them based on a value in app settings. But it looks like the migrations differ.

The solution would be to remove migrations from source control and have users create one themselves, which would have the added benefit of speeding up the database update massively because it won't have to run all the old migrations. I gather though that this is supposedly bad practice.

@avioletheart
Copy link
Contributor Author

oh, are the migrations generated somehow? that's good to know 😅 i am not really familiar with efcore, just changed things until it worked
it's probably possible to have support multiple providers by removing migrations from source control, i will look into it

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