missing requires statement in genpptx.js about officegen HOT 2 CLOSED

Thanks for the missing requirement...
I'll fix that in the next commit.
About PDF and addImage:
I don't know if it's possible to insert PDF inside pptx as is (but I'll
check it) but you can't do that using addImage.

thanks,
Ziv

On Tue, Oct 15, 2013 at 8:39 PM, Seth Pollack [email protected]:

Just hit this, in genpptx.js, line 889.

node_modules/officegen/lib/genpptx.js:889
var image_ext = path.extname ( image_path );
^
ReferenceError: path is not defined

I hit this by trying to insert a pdf file just to see what would happen.
is it possible to stick a pdf directly into an pptx document? or does it
need to be converted to emf or similar first?

Cool library, thanks for sharing it.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/5
.

from officegen.

Thanks.

from officegen.