danvergara commented 2 weeks ago

Add support for ebooks

Description

This PR adds support for some of the most common e-books. To do this I had to embed the ebook-convert binary from the Calibre project. The tool is installed on the Container image.

Fixes #60

Type of change

Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Added more tests and QA'd the new conversions.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

danvergara commented 1 week ago

@ThiagoMowszet @Sirpyerre I'm not done with this one, guys. I appreciate the early review, but I'm far from done adding support for ebooks. When the time to properly review this, I'll move the PR from draft mode to ready for review.

A refactor is dramatically needed but I'm more focused on move things along. When I reach a desired stated, I'll go over the code and build a solution off the most common patterns.

I'll let you know, guys.

ThiagoMowszet commented 7 hours ago

@ThiagoMowszet @Sirpyerre I'm not done with this one, guys. I appreciate the early review, but I'm far from done adding support for ebooks. When the time to properly review this, I'll move the PR from draft mode to ready for review.

A refactor is dramatically needed but I'm more focused on move things along. When I reach a desired stated, I'll go over the code and build a solution off the most common patterns.

I'll let you know, guys.

Thanks for the update @danvergara!

danvergara / morphos