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

mass search #157

Merged
merged 7 commits into from
Mar 12, 2022
Merged

mass search #157

merged 7 commits into from
Mar 12, 2022

Conversation

Robonau
Copy link
Contributor

@Robonau Robonau commented Mar 7, 2022

add mass search
still slow, since it can only grab one source search at a time
for the record, i think this should be implemented server-side (with a WebSocket)

button to go to the search all page:
Screenshot_20220308_194726

(page is empty when first loading)
what the page looks like when you search:
Screenshot_20220308_194847

when the source cant find anything:
Screenshot_20220308_194957

Suwayomi/Suwayomi-Server#209
though that is a server issue

currently cripples server for like 2 mins on use
its slow, really slow
but it don't break the server anymore
@Robonau
Copy link
Contributor Author

Robonau commented Mar 8, 2022

il probably need to fix this after #156 goes through, so its staying a draft for now

@Robonau
Copy link
Contributor Author

Robonau commented Mar 8, 2022

concurrency makes it quite a bit faster, but its still not as fast as tachiyomi
perhaps i'm just using slow sources
Video

@AriaMoradi
Copy link
Member

remove package-lock.json the project uses yarn for managing packages.

@Robonau Robonau marked this pull request as ready for review March 11, 2022 19:24
@AriaMoradi
Copy link
Member

something might be wrong with your yarn.lock all the links have changed

@Robonau
Copy link
Contributor Author

Robonau commented Mar 11, 2022

should be fine now

@AriaMoradi AriaMoradi merged commit 042172a into Suwayomi:master Mar 12, 2022
AriaMoradi added a commit that referenced this pull request Mar 12, 2022
This reverts commit 042172a.
@AriaMoradi AriaMoradi mentioned this pull request Mar 12, 2022
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.

None yet

2 participants