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

Add ACTUAL_PORT environment variable to docker-compose.yml #333

Merged
merged 2 commits into from
Apr 7, 2024

Conversation

psybers
Copy link
Contributor

@psybers psybers commented Apr 7, 2024

It can be difficult to see how to bind Actual server to a different port when using docker compose. This adds a (commented out) environment variable for ACTUAL_PORT to make that more obvious to users.

@trafico-bot trafico-bot bot added the 🚧 WIP label Apr 7, 2024
@psybers psybers changed the title [WIP] Add ACTUAL_PORT environment variable to docker-compose.yml Add ACTUAL_PORT environment variable to docker-compose.yml Apr 7, 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.

Makes sense. Thanks!

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review labels Apr 7, 2024
@MatissJanis MatissJanis merged commit f06bbf9 into actualbudget:master Apr 7, 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 7, 2024
MMichotte pushed a commit to MMichotte/actual-server that referenced this pull request Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Merged Pull Request has been merged successfully
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants