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

Error screen #1375

Closed
wants to merge 11 commits into from
Closed

Error screen #1375

wants to merge 11 commits into from

Conversation

cuong-tran
Copy link
Contributor

@cuong-tran cuong-tran commented Oct 25, 2024

  • A dedicated screen in Settings for error list when updating library
  • Allow to jump to Error screen if click on notification
  • Allow to migrate error entry
  • Create error on each entry updated instead of waiting for the whole updating list to finished
  • Overwrite entry's error if new error happens after updating
  • Clear entry's error if it successfully updated
  • Clear un-relevant errors (entry which was removed from Library) on next update
  • List of errors can jump to top/bottom or next/previous errors group
  • Won't create error file anymore

Close #132

image
image

@MajorTanya
Copy link
Contributor

Your images appear to be broken right now

@AntsyLich
Copy link
Member

https://git.mihon.tech/tachiyomi/tachiyomi/pulls/9795 is what I had (alongside others) reviewed during the tachi days. And I like this implementation better.

@cuong-tran
Copy link
Contributor Author

https://git.mihon.tech/tachiyomi/tachiyomi/pulls/9795 is what I had (alongside others) reviewed during the tachi days. And I like this implementation better.

why don't bring it over?

@Animeboynz
Copy link
Contributor

I was saying I planned on it but then my exams started
.

@AntsyLich
Copy link
Member

Closing in favor of #1381

@AntsyLich AntsyLich closed this Oct 26, 2024
@cuong-tran cuong-tran deleted the error-screen branch October 26, 2024 14:13
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

In app Error list for chapter update errors
5 participants