Skip to content

Comments

Create database if it doesn't exist#300

Merged
lythx merged 1 commit intolythx:devfrom
AroPix:feature/create-database
Feb 11, 2026
Merged

Create database if it doesn't exist#300
lythx merged 1 commit intolythx:devfrom
AroPix:feature/create-database

Conversation

@AroPix
Copy link
Contributor

@AroPix AroPix commented Nov 16, 2025

This creates the database if it doesn't exist (makes it way easier if planning to deploy many servers).
It will crash because the logger tries to use the database before it exists, but it actually creates it.
Not ideal but in the Docker container it restarts itself anyway, which will work then because it initializes on the first try.

@AroPix AroPix force-pushed the feature/create-database branch from 44ffa58 to c806836 Compare November 16, 2025 00:14
@AroPix AroPix changed the base branch from main to dev November 18, 2025 23:53
@AroPix
Copy link
Contributor Author

AroPix commented Nov 18, 2025

gg i rebased this on top of main instead of dev

@AroPix AroPix marked this pull request as draft November 18, 2025 23:54
@AroPix AroPix force-pushed the feature/create-database branch from c806836 to 4309a2f Compare November 24, 2025 13:13
@AroPix AroPix force-pushed the feature/create-database branch from 4309a2f to 8d1eeef Compare November 24, 2025 13:15
@AroPix AroPix closed this Nov 24, 2025
@AroPix AroPix reopened this Nov 24, 2025
@AroPix AroPix marked this pull request as ready for review November 24, 2025 13:30
@wsrvn wsrvn requested a review from lythx November 27, 2025 14:49
Copy link
Owner

@lythx lythx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The crash on startup due to database access is a known issue - this will stop crashing when we fix it. Restart fixes it so it's fine.

@lythx lythx merged commit 9a710e2 into lythx:dev Feb 11, 2026
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