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

Dub mapping Improvements #104

Merged
merged 6 commits into from
Nov 9, 2022
Merged

Dub mapping Improvements #104

merged 6 commits into from
Nov 9, 2022

Conversation

ErrorErrorError
Copy link
Collaborator

@ErrorErrorError ErrorErrorError commented Nov 9, 2022

What kind of change does this PR introduce?
This pr fixes some issues where gogoanime was not returning the correct dub episodes. This also fixes issues where some providers might have bad search functionality when retrieving anime information. This will allow more accurate results when fetching for episodes.

Did you add tests for your changes?
Nope.

If relevant, did you update the documentation?
N/A

Summary
I want to be able to watch the correct episode and if I change providers, to make sure it returns the correct anime information.

Other information
I bumped the version to 1.2.7 and updated changelog.

… dub and subs

- Anilist now finds animeinfo more accurately for dub
- mapping episodes is now possible in the future.
@riimuru riimuru changed the title Dub Improvements Dub mapping Improvements Nov 9, 2022
aidanjuma
aidanjuma previously approved these changes Nov 9, 2022
Copy link
Contributor

@aidanjuma aidanjuma left a comment

Choose a reason for hiding this comment

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

Looks good to me! I've pinged @5H4D0WILA to check as well.

edit: nvm I'm bad hehe - thanks @riimuru!!

src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
@aidanjuma aidanjuma self-requested a review November 9, 2022 22:46
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
@aidanjuma aidanjuma dismissed their stale review November 9, 2022 22:54

Changes made to syntax.

src/providers/meta/anilist.ts Outdated Show resolved Hide resolved
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.

LGTM 👍

@aidanjuma aidanjuma merged commit 42a0d3b into consumet:master Nov 9, 2022
@riimuru riimuru added the enhancement New feature label Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants