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

Fix TOC and Google Play upload issues [Issue #37] #38

Closed
wants to merge 3 commits into from

Conversation

Kietyo
Copy link

@Kietyo Kietyo commented Jan 3, 2019

This commit fixes TOC so that it works with Microsoft Edge as well as Google Play Books. It also fixes the upload issues for Google Play Books.

I fixed the Google Play books stuff by reverse engineering from a epub book that I already knew works with Google Play Books.

These commits also have changes that occurred when I ran "Reformat Code" in the PyCharm IDE that I used.

@Kietyo Kietyo changed the title Fix TOC and Google Play upload issues Fix TOC and Google Play upload issues [Issue #37] Jan 3, 2019
@MichaelGuardian
Copy link
Contributor

Thanks for taking the time on fixing the issue!
Sadly I can't accept the pull request.

It would be nice if you can fix the following issues and resubmit the pull request:

  • Formatting:

Please refrain from changing the formatting without the approval of the repository maintainers. While there isn't a style guide for this project don't change the code of other people without a good reason.

  • Commit message:

Change the commit messages to something meaningful "save for now" is not a good commit message. Also you can reference issues with #isssue_number.

  • Avoid moving code unnecessary:

This makes it hard to review and track changes. Especially if it serves no purpose at all.

These are some general tips you should keep in mind if you're contributing to pubic repositories ;)

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