Skip to content

Fix linter warnings and update dependencies#2313

Merged
K1ngfish3r merged 3 commits into
lnreader:masterfrom
K1ngfish3r:linter
Jul 15, 2026
Merged

Fix linter warnings and update dependencies#2313
K1ngfish3r merged 3 commits into
lnreader:masterfrom
K1ngfish3r:linter

Conversation

@K1ngfish3r

Copy link
Copy Markdown
Collaborator

Checklist

  • Update version code if an existing plugin was modified
  • Test changes in Plugin Playground or the app
  • Reference related issues in the PR body (e.g. Closes #xyz)

Signed-off-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
Signed-off-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
Signed-off-by: K1ngfish3r <26593485+K1ngfish3r@users.noreply.github.com>
@K1ngfish3r K1ngfish3r merged commit cd6a44b into lnreader:master Jul 15, 2026
1 check passed
@K1ngfish3r K1ngfish3r deleted the linter branch July 15, 2026 11:10
@MonomoySama

Copy link
Copy Markdown

It seems to me that you fixed some errors for Lnori plugin? Well, i just installed it to try a bit but couldn't even open a single entry. It just get stuck on load screen and app stuck too, had to close and reopen app

@K1ngfish3r

Copy link
Copy Markdown
Collaborator Author
image

hmm, not sure, worked fine on webtest...

@MonomoySama

Copy link
Copy Markdown
image hmm, not sure, worked fine on webtest...

Ok so i tried Re Zero, and it took nearly 2min to load fully, even then, there was one volume missing(couldn't fetch maybe), so i tried another. In the 3 images, most entries are giving "cant parse page" and those that do give result, always misses one random volume.
Screenshot_2026-07-16-00-35-06-48
Screenshot_2026-07-16-00-34-45-58
Screenshot_2026-07-16-00-32-09-51

@K1ngfish3r

Copy link
Copy Markdown
Collaborator Author

after benching this
2s on desktop
41s on mobile

@MonomoySama

Copy link
Copy Markdown

after benching this 2s on desktop 41s on mobile

Well, surely users are not using it on desktop, so gotta improve it more for mobile, and btw, that cant parse error and misisng volumes? You find the reason?

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