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

Add advanced Library setting to allow library manga's titles to update from source #1182

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FlaminSarge
Copy link
Contributor

@FlaminSarge FlaminSarge commented Sep 1, 2024

This adds a setting under Advanced > Library to allow any entry, not just favorites, to update their titles from source.
It attempts to move the downloaded files as well by renaming the entry's download folder. It does not currently attempt to move downloaded files into an existing folder (if another title exists/conflicts).

A bit of context from the previous change to allow non-favorites to update title from source: 58a871c

Tested cases:

  • Auto-update metadata setting disabled, title update setting enabled: refreshing library does not update to new title, but refreshing from manga details page does
  • Auto-update metadata setting enabled, title update setting enabled: refreshing library does update to new title, as does refreshing from manga details page
  • Title update setting disabled: neither refreshing library nor manga details page updates the title

Images

Image 1 Image 2
Screenshot 2024-08-31 at 5 35 21 PM Screenshot 2024-08-31 at 5 35 53 PM

@FlaminSarge
Copy link
Contributor Author

FlaminSarge commented Sep 1, 2024

I was considering putting this under the "Automatically update metadata" setting under Library settings; unsure whether it should go there or under Advanced, given how it's a slightly more dangerous setting when it comes to Download handling.

I need advice on better download handling for this PR before I move it out of draft, as while a simple rename works for some cases, it doesn't handle the edge case of existing folder nor do I do any cache handling yet.

Another discussion that came up is changing the Auto-update metadata setting (or this setting) to be a multiselect preference with the various metadata fields as options, such that a user can choose to e.g. update Title but preserve Cover (for whatever purpose). Unsure if that's overdoing it here.

i18n/README.md Outdated Show resolved Hide resolved
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