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

feat(dramacool): Add recent-shows and recent-movies routes to DramaCool, trailer field to IMovieInfo, refactor(goku, moviehdwatch): Implement movie fetching by country and genre. Standardize outputs (e.g., remove .replace('SS ', '') from Goku). #574

Merged
merged 5 commits into from
Aug 14, 2024

Conversation

2004durgesh
Copy link
Contributor

What kind of change does this PR introduce?
Added recent-shows and recent-movies routes to Dramacool, introduced a trailer field to IMovieInfo, and implemented movie fetching by country and genre. Additionally, standardized outputs from movie providers (e.g., removed .replace('SS ', '') from Goku).

Did you add tests for your changes?
Yes, I did

If relevant, did you update the documentation?
Yes, I did

Summary
As FlixHQ provides season and episode information, I wanted those changes applied to Goku and MovieHDWatch as well. Additionally, added recent-movies and recent-shows methods to Dramacool, since other movie providers also have these features.

Other information
No

…ol, trailer field to IMovieInfo,

refactor(goku, moviehdwatch): Implement movie fetching by country and genre. Standardize outputs (e.g., remove `.replace('SS ', '')` from Goku).
Copy link
Contributor

@riimuru riimuru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall it looks great! 👍

src/providers/movies/flixhq.ts Outdated Show resolved Hide resolved
@2004durgesh
Copy link
Contributor Author

done fixing indentation 👍

@riimuru riimuru merged commit 3dd0ccb into consumet:master Aug 14, 2024
3 of 5 checks passed
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.

3 participants