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

All current fixes by me #75

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

PeterVoronov
Copy link

Hiif you interested in some fixes - please see the my forkwith small fixes:

  1. Telebot compatible with version 13.5 of the used api
  2. Fix working with extensions in telebot
  3. Fixes in OPDS - authentication is now fully work for me from CoolReader
  4. Cosmetic fix for bookshelf

[*] Fix the "RegexHandler is deprecated. See https://git.io/fxJuV
    for more info" in Telebot.
[*] Change order of handlers to cats the regexp (download command)
    before the common text (book search) in Telebot.
	modified:   management/commands/sopds_telebot.py
[*] Fix extension of downloaded files in mobi and epub format:
    will be some_name.epub instead some_name.fb2.epub, and
    some_name.mobi instead some_name.fb2.mobi.

	modified:   opds_catalog/management/commands/sopds_telebot.py
…e in book list, due to duplicates consolidation
…if configured. It will authomatically put downloaded books on bookshelfs for authenticated users
@ignis32
Copy link

ignis32 commented Nov 25, 2021

Thanks a lot for sharing this, I've used your code instead of original, and your fork worked for me.
Without your fixes nor Coolreader, nor Telegram bot were functional.

@PeterVoronov
Copy link
Author

Thanks a lot for sharing this, I've used your code instead of original, and your fork worked for me. Without your fixes nor Coolreader, nor Telegram bot were functional.

You are welcome ;-)

Somehow it is speedup processing the result of the query ...
- logic of finding doubles moved to the database query level.
- fixed exception on sending the same message;
- some code cosmetic fixes.
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