Skip to content
This repository has been archived by the owner on Feb 16, 2023. It is now read-only.

Added a sentence about python-dotenv format #1222

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

flofeld
Copy link

@flofeld flofeld commented Aug 9, 2021

Something like this would have helped me to get the configuration right without debugging

Something like this would have helped me to get the configuration right without debugging
@@ -455,6 +455,8 @@ Add the configuration variables to the environment of the webserver (alternative
put the configuration in the ``docker-compose.env`` file) and add the additional
services below the webserver service. Watch out for indentation.

Make sure tu use the correct format `PAPERLESS_TIKA_ENABLED = 1`so python_dotenv can parse the statement correctly.

Choose a reason for hiding this comment

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

There're two typos here: "tu" and a missing space right next to the last backtick.

The latter one results in ill-rendered text.

image

@iwconfig
Copy link

Also, what format did you use prior to this one? PAPERLESS_TIKA_ENABLED = '1'? Maybe you should point that out. Come to think of it, issues of wrongly declared variables affect just about any variable, thus you should rather point this out instead of just only PAPERLESS_TIKA_ENABLED.

For example the fact that docker and docker-compose does not follow the same standard as shell regarding the .env file, and maybe a reference to this SO post for context when declaring variables in YAML (or a better source if you have one, I just did a quick google search).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants