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

Fix yavka.net provider by passing all form parameters. Improve speed #2835

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

screamerbg
Copy link

At the moment (25 Jan 2025) the Yavka.net provider is broken due to recent changes on their website (like reordered form fields etc)

This PR addresses the following:

  • Fixes the Yavka.net provider by passing all form parameters
  • Improves the use of cached files by adding a cache check while iterating through search results, effectively avoiding an unnecessary call to their website
  • Process the newest subtitles first in the search list
  • Process 50 search entries instead of 25 previously (which caused valid subtitle results to be missed)

The above significantly boost the provider speed, especially in cases like TV series subs

@morpheus65535
Copy link
Owner

Rebase on dev branch and I'll merge this ASAP. Thanks!

@screamerbg screamerbg changed the base branch from master to development January 27, 2025 13:21
@screamerbg
Copy link
Author

Rebase done

@morpheus65535 morpheus65535 merged commit 99d9691 into morpheus65535:development Jan 27, 2025
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