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

Create files in final folder only when the scan is finished #55

Closed
whysthatso opened this issue Aug 28, 2020 · 2 comments · Fixed by #59
Closed

Create files in final folder only when the scan is finished #55

whysthatso opened this issue Aug 28, 2020 · 2 comments · Fixed by #59
Assignees
Milestone

Comments

@whysthatso
Copy link

heya,
i'm using scanservjs as an input scanner for my edms papermerge.

scanservjs's output folder is papermerge's input folder.

unfortunately, as soon as scanservjs creates the empty file to be filled with data upon commencement of a scan, papermerge's worker can already grab the file and move it out of the folder.

would it be possible to finish the file in a temporary location and move it into the output folder thereafter? maybe that can be configurable, or system-tmp.

@JnCrMx
Copy link
Contributor

JnCrMx commented Sep 7, 2020

I did a quick (and rather dirty) solution. To be honest I'm not sure if it's worth merging into master.

But for you it might be quite useful.

@sbs20 sbs20 added this to the v2.0.0 milestone Sep 12, 2020
@sbs20 sbs20 changed the title feature: do not create files in final folder until the scan is finished Create files in final folder only when the scan is finished Sep 13, 2020
@sbs20 sbs20 self-assigned this Sep 13, 2020
@sbs20 sbs20 mentioned this issue Sep 13, 2020
@sbs20 sbs20 linked a pull request Sep 13, 2020 that will close this issue
@sbs20 sbs20 closed this as completed in #59 Sep 13, 2020
@whysthatso
Copy link
Author

thanks all for the good work 👍

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

Successfully merging a pull request may close this issue.

3 participants