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

Library Update Errors Screen #1381

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

Animeboynz
Copy link
Contributor

@Animeboynz Animeboynz commented Oct 26, 2024

Migrated PR from Tachiyomi#9795

Closes #132

Things left to do

  • Null Crash on Untrusted Extensions
  • Add icons for untrusted extensions
  • Investigate key was already used crash
  • Fix Key was already used crash
  • Revisit ExtendedFloatingActionButton
  • i18n of Exception Strings
  • Questionable Fractions
  • All failed updates gets reset on new update
  • Failed update log does not get removed when updating individual manga (only global or category)
  • Certain similar errors not grouped together

Elaboration

Key was already used crash

Key was already used crash

Issue Found - 11/03/2025

Happens when you have a manga in more than 1 category.

2025-03-11 15:31:39.190  5994-5994  DupKey                  app.mihon.dev                        E  Manga ID: 153, Manga Name: Ojousama Wa Nigedashita, Manga Source: 1111111111111111111, Manga URL: /manga-ab123456, UnknownHostException: Unable to resolve host "shittygenericscanlators.com": No address associated with hostname, Category(id=2, name=Up-to-date, order=1, flags=28)
2025-03-11 15:31:39.190  5994-5994  DupKey                  app.mihon.dev                        E  Manga ID: 153, Manga Name: Ojousama Wa Nigedashita, Manga Source: 1111111111111111111, Manga URL: /manga-ab123456, UnknownHostException: Unable to resolve host "shittygenericscanlators.com": No address associated with hostname, Category(id=9, name=Translation Abandoned, order=8, flags=80)

ExtendedFloatingActionButton

ExtendedFloatingActionButton

These don't seem to exist anymore, so the fab is always extended.

import tachiyomi.presentation.core.util.isScrolledToEnd
import tachiyomi.presentation.core.util.isScrollingUp
floatingActionButton = {
      AnimatedVisibility(
          visible = state.items.isNotEmpty(),
          enter = fadeIn(),
          exit = fadeOut(),
      ) {
          ExtendedFloatingActionButton(
              text = { Text(text = stringResource(R.string.label_help)) },
              icon = { Icon(imageVector = Icons.Outlined.HelpOutline, contentDescription = null) },
              onClick = { uriHandler.openUri("https://tachiyomi.org/help/guides/troubleshooting") },
              //expanded = failedUpdatesListState.isScrollingUp() || failedUpdatesListState.isScrolledToEnd(),
          )
      }
},
Fractions Error

Fractions Error

Seems to be caused by Multi Language Extentions.

Eg. Ext-ENG had 16 entries with 16 failed. Ext-Multi has 4 entries with 4 failed. The fails of multi get added to the ENG without it updating the total count showing 20/16

All failed updates gets reset on new update

All failed updates gets reset on new update

Triggering any category update at all, wipes the log of errors.

Ideally it should only reset for a manga when that manga is updated again.

Eg. You have 50 entries with errors. you update a category with 1 entry in it. All error logs gets reset.

Certain similar errors not grouped together

Certain similar errors not grouped together

Screenshot_20250312_092057_Mihon

@Animeboynz Animeboynz force-pushed the UpdateErrorsScreen branch 8 times, most recently from 7e3ee1b to 531539b Compare October 26, 2024 11:07
@AntsyLich AntsyLich mentioned this pull request Oct 26, 2024
@cuong-tran
Copy link
Contributor

add more screenshot pls, I saw 1 in Discord, this looks nice

@Animeboynz
Copy link
Contributor Author

add more screenshot pls, I saw 1 in Discord, this looks nice

Here ya go. Was just too lazy to censor stuff while half asleep. 🤣
It is supposed to say the error reason under the title when ungrouped but string need to be fixed.

Screen_Recording_20241027_063400_Mihon_1.mp4

@Animeboynz Animeboynz force-pushed the UpdateErrorsScreen branch 2 times, most recently from c33a3a7 to b43b0f8 Compare October 26, 2024 23:05
@cuong-tran
Copy link
Contributor

should close this #132

@AntsyLich
Copy link
Member

Changed state to draft

@imkunet
Copy link

imkunet commented Feb 26, 2025

  • Null Crash on Untrusted Extensions
  • Add icons for untrusted extensions
  • Investigate key was already used crash
  • Revisit ExtendedFloatingActionButton
  • i18n of Exception Strings
  • Questionable Fractions
  • All failed updates gets reset on new update

@AntsyLich
Copy link
Member

All failed updates gets reset on new update

Imo only the successful ones should be removed

@Animeboynz
Copy link
Contributor Author

Animeboynz commented Feb 26, 2025

That checkbox was intended for removing that behavior btw.
I really hate the current behavior, update one category and everything gets reset

@Animeboynz Animeboynz force-pushed the UpdateErrorsScreen branch 2 times, most recently from 1a956ec to 9925f4d Compare March 11, 2025 04:14
Animeboynz and others added 2 commits March 12, 2025 09:44
Co-Authored-By: Programmer-0-0 <67505416+programmer-0-0@users.noreply.github.com>
@Animeboynz Animeboynz force-pushed the UpdateErrorsScreen branch 2 times, most recently from 02e8f6f to c1fc2f7 Compare March 11, 2025 20:52
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.

In app Error list for chapter update errors
4 participants