-
Notifications
You must be signed in to change notification settings - Fork 261
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
Zoro fetchEpisodeSources and fetchEpisodeServers fixed #594
base: master
Are you sure you want to change the base?
Conversation
): IExtendedEpisodeServer | undefined { | ||
const serverMap: { [key: string]: string } = { | ||
[StreamingServers.VidStreaming]: 'HD-1', | ||
[StreamingServers.VidCloud]: 'HD-2', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The function should accept multiple values like 'VidCloud' and 'VidStreaming' because the user won't know they are labeled as 'HD-1' or 'HD-2' to select them. It should also be case-insensitive. The function below should updated accordingly.
throw err; | ||
} | ||
}; | ||
// override fetchEpisodeSources = async ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this commented block
What kind of change does this PR introduce?
= i fixed zoro fetchEpisodeSources and also added fetchEpisodeServers
Did you add tests for your changes?
= yes
If relevant, did you update the documentation?
= no already updated
Summary
Other information