Skip to content
This repository has been archived by the owner on Apr 15, 2020. It is now read-only.

Ensure the image paths are sorted naturally (numerically) so that 10 … #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RoyHP
Copy link

@RoyHP RoyHP commented Oct 8, 2018

…and 11 come after 1-9

My fix for #45

@rainabba
Copy link

rainabba commented Feb 7, 2019

@mooz Might you be able to give us an ETA on this? This is critical to my use of the library since nearly all our PDFs will have >10 pages (100's will be common).

@RoyHP Thanks for this. Any reason I shouldn't use your fork/branch for now?

@RoyHP
Copy link
Author

RoyHP commented Mar 19, 2019

@rainabba You're welcome. Feel free to use it from my repo, it's not going anywhere.

"pdf-image": "git+https://github.com/RoyHP/node-pdf-image.git#432a07866759e73bdfc6942e5f32ee49b874adf2",

@roest01
Copy link
Contributor

roest01 commented Mar 19, 2019

the maintenance of this repo is a shame 🙁

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.

3 participants