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

Improve systemchecks #83

Closed
R0Wi opened this issue Dec 9, 2021 · 1 comment · Fixed by #289
Closed

Improve systemchecks #83

R0Wi opened this issue Dec 9, 2021 · 1 comment · Fixed by #289
Labels
documentation Improvements or additions to documentation enhancement New feature or request pinned

Comments

@R0Wi
Copy link
Contributor

R0Wi commented Dec 9, 2021

Since one problem in #79 was the missing ocrmypdf-commandline-tool, we should at least update the appstore docs to point to our README for further installation instructions. Also we could try to add a check inside "Security & setup warnings" section of Nextcloud.

EDIT: adding checks to Nextcloud seems currently not possible. What we could do is introduce a general check before saving the workflow at https://github.com/R0Wi/workflow_ocr/blob/master/lib/Operation.php#L82.

@R0Wi R0Wi added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 9, 2021
@R0Wi R0Wi added the pinned label Feb 28, 2023
@R0Wi
Copy link
Contributor Author

R0Wi commented Dec 24, 2024

R0Wi added a commit that referenced this issue Dec 29, 2024
* Do some basic check if OCRMyPDF CLI is working
* Fixes #83
R0Wi added a commit that referenced this issue Dec 29, 2024
* Do some basic check if OCRMyPDF CLI is working
* Fixes #83
@R0Wi R0Wi closed this as completed in #289 Dec 29, 2024
@R0Wi R0Wi closed this as completed in 7bddb53 Dec 29, 2024
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 enhancement New feature or request pinned
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant