Open MP70 opened 1 year ago
Has there been any follow up regarding this? I seem to be in a similar position.
This project seems pretty dead IMO. I ended up not using it. Feel free to use my code above as a starting point if you'd like, consider it MIT licensed.
Issue with loading PowerPoint file from buffer
The
save()
method allows you to specify either a callback (passing buffer) or file path, but when loading a PowerPoint file, you must pass a file path. This can be pretty inconvenient in cases where you already have the PowerPoint file contents in a buffer.Proposed solution
To address this issue, can we please add support for loading from a buffer as well as a file path.
I've quickly scanned your code, and at first glance it looks like it's as simple as changing
loadExistingPPTX()
: https://github.com/heavysixer/node-pptx/blob/7e137b5743b2ea46f9f8c7ef3d48ecc4a6d7ff05/lib/presentation.js#L99-L103 To something like:Obviously we'd also need to change the variable name to
templateFile
or similar and any other housekeeping as needed.Would you accept a PR from me along these lines to add this feature?