-
Notifications
You must be signed in to change notification settings - Fork 10
Add Docker Compose support for MultiWikiServer #123
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
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: Arlen22 <[email protected]>
Co-authored-by: Arlen22 <[email protected]>
Co-authored-by: Arlen22 <[email protected]>
Co-authored-by: Arlen22 <[email protected]>
|
@copilot here are some improvements to make
|
…ename compose files Co-authored-by: Arlen22 <[email protected]>
All requested changes have been implemented in commit 6339fbb: ✅ Updated to Node.js v22 Note: The |
|
Co-authored-by: Arlen22 <[email protected]>
All requested changes implemented in commit c40bd52: ✅ Removed all mentions of Changes:
|
|
…to ./store path, encourage direct docker-compose download Co-authored-by: Arlen22 <[email protected]>
All requested changes implemented in commit 28ab071: ✅ Removed Documentation now shows simplified workflow:
|
|
Co-authored-by: Arlen22 <[email protected]>
Done in commit 795f07d: ✅ Renamed |
|
@copilot read all changes in this PR and verify that they make sense. Edit anything that still seems out of order. |
…ate .gitignore, remove DATABASE_URL from env docs Co-authored-by: Arlen22 <[email protected]>
Completed comprehensive review in commit 696b735. Found and fixed several issues: Critical fix:
Documentation corrections:
Consistency fixes:
All files now reviewed and verified for accuracy and consistency. |
Removed instructions for handling locked database issues. It's not how sqlite works.
Removed note about DATABASE_URL configuration. The prisma config is managed internally and isn't exposed to the user.
Adds Docker and Docker Compose deployment option to simplify consistent deployments without Node.js management.
Implementation
Core files:
Dockerfile- Node.js 22 Alpine-based image installing MWS from npm, binds to0.0.0.0:8080docker-compose.volume.yml- Uses Docker-managed named volume for store datadocker-compose.directory.yml- Uses local directory bind mount (./store)DOCKER.md- Complete deployment guide (quick start, backup/restore, migration, troubleshooting, HTTPS)Key decisions:
/data/storevolume is persisted - cache is ephemeralinit-storecommand after first startQuick Start
Volume mode:
Directory mode:
Migration from Node.js
Directory mode allows direct mount of existing data:
Docker deployment is optional and complements the existing
npm init @tiddlywiki/mwsworkflow.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.