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(|dramacool | multimovies): change in scraping logic for dramacool due to change in baseUrl, update baseURL of multimovies and improve server fetching logic. #651

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

2004durgesh
Copy link
Contributor

What kind of change does this PR introduce?
This PR refactors the scraping logic for Dramacool to accommodate changes in its base URL and HTML structure, updates the base URL for MultiMovies, and improves the server fetching logic.

Did you add tests for your changes?
Yes

If relevant, did you update the documentation?
No

Summary

  • Dramacool: Updated the scraping logic to work with the new base URL and modified HTML structure. Note that the new base URL does not include some popular dramas (e.g., Vincenzo), which may result in these titles not showing up.
  • MultiMovies: Updated the base URL and enhanced the server fetching mechanism for improved reliability and performance.

Other information

  • I apologize for the mistake in my last PR that made the whole of Consumet go kaboom.
  • For reference, I've kept the old server fetching logic intact within the Dramacool provider code, so it can be revisited if needed.

… due to change in baseUrl, update baseURL of multimovies and improve server fetching logic.
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.

1 participant