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

@Indigo183
Copy link

Indigo183 commented Jan 15, 2025

I was looking to add MySQL support for Docker (mainly because it supports ARM processors) and stumbled across this, would this also support ARM?

@Indigo183
Copy link

And if I wanted to run this using Docker, what of docker-compose.yml do I need to edit? As far as I can tell, only this?
image

@avioletheart
Copy link
Contributor Author

yeah postgres runs on arm. there would be more docker-compose changes needed than that but not much more, i think?
sorry, i haven't had time to look at this recently

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