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

FR: add support for other read-it-later services (omnivore,shiori) #10406

Open
Amolith opened this issue May 9, 2023 · 14 comments
Open

FR: add support for other read-it-later services (omnivore,shiori) #10406

Amolith opened this issue May 9, 2023 · 14 comments

Comments

@Amolith
Copy link

Amolith commented May 9, 2023

Omnivore (GitHub) is an open source read-it-later service similar to wallabag, which KOReader already has support for. I'd like to see support added for Omnivore as well; its UI is much more pleasant than wallabag's and they have a promising roadmap.

If this would be welcome as a PR, I'll take a look when I have time over the summer.

@Frenzie
Copy link
Member

Frenzie commented May 9, 2023

If it works sufficiently similar maybe the Wallabag plugin can be generalized. I'd be less enthused about having another plugin that's extremely similar but different, anyway. ;-)

@Frenzie Frenzie added the Plugin label May 9, 2023
@pazos
Copy link
Member

pazos commented May 9, 2023

A bit dissapointing https://github.com/omnivore-app/omnivore#how-to-deploy-to-your-own-server

@aqxa1
Copy link

aqxa1 commented Jun 23, 2023

There's also shiori which is much faster than wallabag on my system (at least with the official docker container) and deployment is simple.

EDIT: Although I'm not sure how usable it is API wise.

EDIT2: Here are the API docs.

@Amolith
Copy link
Author

Amolith commented Jun 25, 2023

Looking around some more, neither Omnivore nor Shiori export EPUB files. Wallabag definitely does and I'm guessing that the current KOReader plugin just lets wallabag do the hard conversion work and it downloads the result.

Adding Omnivore/Shiori support would make the plugin much more complex because it would have to do all that conversion itself. I don't have the time or motivation to implement this, so I'll just stick with wallabag for now ^^'

@aqxa1
Copy link

aqxa1 commented Jul 7, 2023

A PR for epub generation is close to being merged to shiori. So hopefully it becomes realistic once that happens.

@Frenzie
Copy link
Member

Frenzie commented Jul 7, 2023

Creating an EPUB in KOReader isn't much work — or rather, the work's already been done before.

That being said, in those linked Shiori API docs I don't even see how to get the content, just the URL? Now that could complicate matters significantly, given how authentication can also be an important part of these systems, cf. https://github.com/wallabag/wallabag/pull/2317.

@Monirzadeh
Copy link
Contributor

Monirzadeh commented Jul 7, 2023

I start working on shiori ebook Generation to link that to KOReader later. For now, shiori is under heavy API refactor on go-shiori/shiori#497.
But for now the request should send just the ID to the server, and it generates EPUB for that specific bookmark.
After go-shiori/shiori#497 i try update API section for that epub generate.
For work better with koreader is there any specific feature to be crucial?
As I know it, just need to log in API and request EPUB, (maybe modify tag or request specific tag) in shiori side.
For now, shiori get content and create a clean version with go-readability. We use that result for build ebook.

@Frenzie
Copy link
Member

Frenzie commented Jul 7, 2023

Other than actually getting content in any form at all I don't see anything obviously missing in the API docs.

For PDF you'd generally want to download the actual PDF instead of some text extracted EPUB.

@Frenzie
Copy link
Member

Frenzie commented Jul 7, 2023

Edit would ideally also include read status though; I don't see that listed.

@Monirzadeh
Copy link
Contributor

Monirzadeh commented Jul 7, 2023

Edit would ideally also include read status though; I don't see that listed.

There is some discussion about how should handle read status go-shiori/shiori#343

For now, we have PDF on disk, but not return in API. I plan to return actual PDF file when I'm starting work on update API for eink device. But it is on hold until go-shiori/shiori#497 done.

@pazos pazos changed the title FR: add support for Omnivore, an open source read-it-later service FR: add support for other read-it-later services (omnivore,shiori) Jul 7, 2023
@Monirzadeh
Copy link
Contributor

hi.
how should i place my plugin beside other plugin?
if i use something like this

function Shiori:addToMainMenu(menu_items)
    menu_items.shiori = {
        text = _("Shiori"),
        sub_item_table = {

koreader place Shiori here
1


but i want place shiori here
2

@pazos
Copy link
Member

pazos commented Jan 22, 2024

@Monirzadeh: you'll need to populate sorting_hint with the string that defines a submenu. You can get all the strings in https://github.com/koreader/koreader/blob/master/frontend/ui/elements/reader_menu_order.lua and related files.

@Monirzadeh
Copy link
Contributor

Monirzadeh commented Jan 23, 2024

@pazos
i add sorting_hint = "tools", and it move to the tools menu but after More tools in second page.
can i move that to the first page?
for example after wallabag?
3

note: i add shiori in https://github.com/koreader/koreader/blob/master/frontend/ui/elements/reader_menu_order.lua after wallabag too, but it is not work

@pazos
Copy link
Member

pazos commented Jan 23, 2024

@pazos i add sorting_hint = "tools", and it move to the tools menu but after More tools in second page. can i move that to the first page? for example after wallabag? 3

note: i add shiori in https://github.com/koreader/koreader/blob/master/frontend/ui/elements/reader_menu_order.lua after wallabag too, but it is not work

Your guess is good. If you want to place the menu entry at a fixed position you need to edit the menu file and ignore sorting hint.

Editing reader menu order will affect the menu of the reader. You need to do ir also on the file manager menu order.

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

No branches or pull requests

5 participants