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.

Sign up for GitHub

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

Jump to bottom

New Year Bugs #2513

Merged
merged 12 commits into from
Jan 3, 2024
Merged

New Year Bugs #2513

merged 12 commits into from
Jan 3, 2024

Conversation

majora2007
Copy link
Member

Added

  • Added: (Kavita+) Added the ability to view the provider of a recommendation on the card itself and in the preview drawer.

Changed

  • Changed: When opening up an external series review, show the source of the review.
  • Changed: Streamlined the design slightly for a user's personal review
  • Changed: There is now inline form validation on the User Review Modal
  • Changed: Removed the tagline from user/external reviews.
  • Changed: (Scanner) When a series has a single special and a total count of 1 or less, assume it's completed

Fixed

  • Fixed: Fixed a bug where user reviews wouldn't be rendered on the UI if they didn't have a tagline, even though it's optional.
  • Fixed: Ensure my own review orders in front of other local user reviews
  • Fixed: When a user resets the API key, tell connected clients to refresh their account details
  • Fixed: Moved bulk operations bar back to the top of the page where it belongs on series detail

Removed

@majora2007
When opening up an external series review, show the source of the rev…
4621acc 
…iew.
@majora2007
Fixed a bug where user reviews wouldn't be rendered on the UI if they…
1ea02c5 
… didn't have a tagline, even though it's optional.
@majora2007
Streamlined the design slightly for a user's personal review and some…
48d3e09 
… light css changes.
@majora2007
Added the ability to view the provider of a recommendation on the car…
c676302 
…d itself and in the preview drawer.
@majora2007
Cleaned up the own user's review card.
70aa309
@majora2007
Code cleanup
0ec9856
@majora2007
Added some inline form validation on the add/edit review modal.
2de109a
@majora2007
Removed the ability to set the tagline and removed the visualization …
ae5e76d 
…of it from External Reviews. This is to streamline and make the system better.

Removed the ability to scrobble reviews as the conditions are strict and doesn't make much sense to do it from Kavita. If notes was possible to write to, I would switch to that.
@majora2007
Ensure that my review always takes precedence over other local users.
3df4235
@majora2007
When we reset an API key, tell connected clients to refresh their acc…
8724a80 
…ount information.
@majora2007
When a series has only a single file and it's a special and either a …
16ecaad 
…total count of 1 or none, then set publication as complete, as it's very likely a single series/book.
@majora2007
Moved bulk operations bar on series detail to the top of the screen, …
c494320 
…where it belongs.
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jan 3, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@majora2007 majora2007 merged commit 5dfcccb into develop Jan 3, 2024
7 checks passed
@majora2007 majora2007 deleted the bugfix/new-years-bugs branch January 3, 2024 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Kavita+ Removal of Review Scrobbling
1 participant
@majora2007