Skip to content
This repository was archived by the owner on Jul 27, 2024. It is now read-only.

Added PDF conversion capabilities and simplified conversion script#17

Closed
InputObject2 wants to merge 1 commit intoBookStackApp:mainfrom
InputObject2:main
Closed

Added PDF conversion capabilities and simplified conversion script#17
InputObject2 wants to merge 1 commit intoBookStackApp:mainfrom
InputObject2:main

Conversation

@InputObject2
Copy link

I ended up reworking the original script quite a bit since I wanted to put the different functions on their own to simplify adding more parsers in the future if needed.

I also added a java check since pdf2html requires java to be in the path to be able to run the .jar for conversion.

There's also some logging on import.

I'm planning to add another modification to this script to parse a folder (if that's what is provided instead of a file name) and upload all the supported files it finds in it.

@ssddanbrown
Copy link
Member

Thanks for offering these changes @InputObject2 but I'm not really looking to increase the scope or functionality of these scripts much wider to keep maintenance reasonable, they're really here to provide some simple starting points or examples for various scenarios.

Feel free though to extract your more developed work into it's own repository for it to then be added to the "Community Projects & Scripts" list in the project readme.

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