Skip to content
This repository was 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
denilsonsa wants to merge 1 commit intojonaswinkler:masterfrom
denilsonsa:patch-3
Open

Tell users about multiple OCR languages in the install script#1374
denilsonsa wants to merge 1 commit intojonaswinkler:masterfrom
denilsonsa:patch-3

Conversation

@denilsonsa
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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.

2 participants