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

docs: boolean value in docker-compose examples not deserializable in env variables #4130

Merged
merged 4 commits into from
Sep 11, 2024

Conversation

ursuscamp
Copy link
Contributor

@ursuscamp ursuscamp commented Aug 31, 2024

What type of PR is this?

  • documentation

What this PR does / why we need it:

Booleans values in docker-compose environment variables are not deserializable and results in an error like this

ERROR: The Compose file './docker-compose.yaml' is invalid because:
services.mealie.environment.ALLOW_SIGNUP contains false, which is an invalid type, it should be a string, number, or a null

Which issue(s) this PR fixes:

N/A

Copy link
Collaborator

@michael-genson michael-genson left a comment

Choose a reason for hiding this comment

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

Thanks! I noticed it works fine in mine without quotes, but I think it's because I use list syntax instead of map syntax? Maybe it's undefined behavior? Who knows, I can barely find docs about it.

Would we be able to use double quotes? Only because we already have the port in double quotes.

@michael-genson michael-genson added the documentation Improvements or additions to documentation label Sep 6, 2024
@hay-kot hay-kot enabled auto-merge (squash) September 11, 2024 22:38
@hay-kot hay-kot merged commit dda6f29 into mealie-recipes:mealie-next Sep 11, 2024
13 checks passed
boc-the-git pushed a commit to boc-the-git/mealie that referenced this pull request Sep 28, 2024
…env variables (mealie-recipes#4130)

Co-authored-by: Hayden <64056131+hay-kot@users.noreply.github.com>
boc-the-git pushed a commit to boc-the-git/mealie that referenced this pull request Sep 28, 2024
…env variables (mealie-recipes#4130)

Co-authored-by: Hayden <64056131+hay-kot@users.noreply.github.com>
Choromanski pushed a commit to Choromanski/mealie that referenced this pull request Oct 1, 2024
…env variables (mealie-recipes#4130)

Co-authored-by: Hayden <64056131+hay-kot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants