-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Possibility to rate from book page without editing #3085
base: master
Are you sure you want to change the base?
Conversation
thank you for working on this. I've applied the PR to my installation, but I don't see any rating option on the book detail page. |
Thanks for your feedback |
@AnonTester Except that, everything working fine on my side |
Thanks, this looks better now, however, I get the following error when trying to change the rating of a book (tested both with a book that doesn't have any rating yet and with one that has an existing rating): 500 Internal Server Error
That's with the patch applied to the docker of 0.6.23 Beta - 02d2c2b - 2024-07-11T20:30:35+02:00 I had to adjust the patch to not overwrite my metadata.db though. |
Hmmm this is what I fixed |
I'm using the linuxserver/calibre-web:nightly docker image and applied the patch from this PR (https://github.com/janeczku/calibre-web/pull/3085.patch) to it after removing the metadata part from the patch. The docker image is based on the latest commit from 5 days ago. workflow:
my docker compose file just for reference:
|
Strange cause the fix concening the error 500 is include in this patch |
same code in your release file and the patch here. I did some light testing and changing line 310 in helper.py to "if not book.ratings:" helps some, but I got some other errors and now got a book where the rating cannot be unset and getting error "Oops! Database Error: UNIQUE constraint failed: books_ratings_link.book, books_ratings_link.rating." I'll do some more testing tonight and get back to you with my results. |
Thanks, I also found this error and fixed it. Now I cannot get the Error 500 anymore but I cannot reproduce your new error concerning the Db constraint |
Fix rating
Not sure for the right CSS file to edit