Skip to content
This repository was archived by the owner on Feb 10, 2025. It is now read-only.

Make running app.js cwd agnostic, extend config to allow changing dataDir #341

Merged

Conversation

matt-fidd
Copy link
Contributor

@matt-fidd matt-fidd commented Apr 16, 2024

Fixes #339 and fixes #340

@matt-fidd matt-fidd force-pushed the extend-config-fix-migrations branch from 3d9c86c to 0d1c08c Compare April 16, 2024 20:56
@matt-fidd matt-fidd changed the title Make running app.js cwd agnostic, extend config to allow changing dataDir [WIP] Make running app.js cwd agnostic, extend config to allow changing dataDir Apr 16, 2024
@matt-fidd matt-fidd force-pushed the extend-config-fix-migrations branch from 0d1c08c to 377ff30 Compare April 16, 2024 20:58
@matt-fidd matt-fidd changed the title [WIP] Make running app.js cwd agnostic, extend config to allow changing dataDir Make running app.js cwd agnostic, extend config to allow changing dataDir Apr 16, 2024
Copy link
Member

@MatissJanis MatissJanis left a comment

Choose a reason for hiding this comment

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

Thanks!

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review labels Apr 19, 2024
@MatissJanis MatissJanis merged commit 44c7b4e into actualbudget:master Apr 19, 2024
6 checks passed
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✅ Approved Pull Request has been approved and can be merged labels Apr 19, 2024
@matt-fidd matt-fidd deleted the extend-config-fix-migrations branch April 19, 2024 20:38
joewashear007 pushed a commit to joewashear007/actual-server that referenced this pull request Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
✨ Merged Pull Request has been merged successfully
Projects
None yet
2 participants