Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Google Play Books + Zip Archive Fix #2

Merged
merged 1 commit into from
May 10, 2016

Conversation

haroldtreen
Copy link
Contributor

Hi @kcartlidge !

nodepub had been creating ebooks for me that worked in iBooks and Calibre but no in Google Play Books. There's an upload feature, and attempting to upload a book would yield an error.

image

I had also noticed that the zip seemed to have a weird structure when I inspected it in my IDE.

image

Looking through the code, I found null folders were being appended to the archive. I removed that line and it fixed the structure issue.

image

However, I still wasn't able to add books to Google Play. I inspected the created book with Calibre and searched for bugs. Calibre suggested <base href='.' /> was pointing to something that did not exist.

image
image

So I removed that. And now created books can be uploaded to Google Books! 😄

Let me know if these changes look good or if there's any concerns 👍 .

@kcartlidge kcartlidge merged commit 1930fec into kcartlidge:master May 10, 2016
@kcartlidge
Copy link
Owner

Hi Harold.

That looks good; I appreciate the input and have merged the pull request, bumped the version number and re-published to npm.

Thanks!

@haroldtreen
Copy link
Contributor Author

Awesome! Happy to contribute :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants