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

Tell users about multiple OCR languages in the install script #1374

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

denilsonsa
Copy link

This valuable piece of information was only mentioned in the docs: https://paperless-ng.readthedocs.io/en/latest/configuration.html#ocr-settings

It would be good to have it in the install script, because that's where/when the user is explicitly asked to choose a language.

This valuable piece of information was only mentioned in the docs: https://paperless-ng.readthedocs.io/en/latest/configuration.html#ocr-settings

It would be good to have it in the install script, because that's where/when the user is explicitly asked to choose a language.
@denilsonsa
Copy link
Author

denilsonsa commented Oct 11, 2021

Hmm… I just noticed this will break line 297, which automatically sets PAPERLESS_OCR_LANGUAGES. Maybe it should be replaced with this?

echo "PAPERLESS_OCR_LANGUAGES=${OCR_LANGUAGE//+/ }"

(Feel free to do this change, or whatever change, by yourself, no need to wait for me in this PR.)

@mweimerskirch
Copy link
Contributor

I prepared a more complete fix some time ago: #1167

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.

None yet

2 participants