`D` attribute is missing from zipped directories

[sindresorhus]

`D` attribute is missing from zipped directories

[isochronous]

Well, after looking at the jsZip documentation, it seems like the .file call should be sufficient, as the .folder method returns a new jsZip instance for chaining, which doesn't really solve the problem as far as I know. I'm going to open an issue with the jsZip library itself.

[dduponchel]

The .folder method creates a new entry in the zip file (flagged as a directory) and returns a new instance for chaining. Calling .folder instead of .file will be enough in this case.

[isochronous]

I don't know whether the fix is as simple as simply switching between .folder and .file based on the results of .isDirectory, but if you think it is, I'll give it a shot. If it's more complex than that, I'll be happy to try out any proposed fix you might come up with.

[isochronous]

Turns out JSZip had only just been updated with a "fix" that caused this problem. I'm working with the JSZip author to provide a fix.

[isochronous]

Could you please publish this new version to npm? This is kind of a sticking point for an active project I'm working on.

[sindresorhus]

@isochronous Yes, when #31 is resolved. Review of that PR welcome.

If you can't wait you can easily depend on a git SHA. See npm docs.

[isochronous]

I had no idea you could do that with npm. That's awesome, thank you!