Skip to content

All current fixes by me#75

Open
PeterVoronov wants to merge 16 commits into
mitshel:masterfrom
PeterVoronov:pv-current
Open

All current fixes by me#75
PeterVoronov wants to merge 16 commits into
mitshel:masterfrom
PeterVoronov:pv-current

Conversation

@PeterVoronov

Copy link
Copy Markdown

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

ignis32 commented Nov 25, 2021

Copy link
Copy Markdown

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
Copy Markdown
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