Flowise icon indicating copy to clipboard operation
Flowise copied to clipboard

Published 20 hours ago • verified publisher icon FlowiseAI

Bugfix/pdf loader add legacy option

Open HenryHengZJ opened this issue 1 year ago • 4 comments

HenryHengZJ avatar Jun 14 '23 12:06 HenryHengZJ

Hi Henry, could you also add the docker dependencies to the ./docker/Dockerfile too as part of this PR, the one that just npm installs flowise? With an update to the dockerhub image. Cheers.

heresandyboy avatar Jun 15 '23 09:06 heresandyboy

hey @heresandyboy I've also updated the file in docker/Dockerfile, do you have additional things you want to add?

HenryHengZJ avatar Jun 15 '23 13:06 HenryHengZJ

hey @heresandyboy I've also updated the file in docker/Dockerfile, do you have additional things you want to add?

Nothing else I can think of for this one. Will you keep the toggle in for 'legacy' parser or make it default - if it is more capable than the other parser? Or perhaps use some error handling to try one, then the other.

heresandyboy avatar Jun 15 '23 14:06 heresandyboy

so far I haven't seen much of the issue there, think will keep it as legacy toggle for now

HenryHengZJ avatar Jun 15 '23 15:06 HenryHengZJ