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

Update Kitsu GraphQL and Media URLs to New Domain #576

Merged

Conversation

user70853572305239875909542835673652937
Copy link
Contributor

  • Updated Kitsu GraphQL and media URLs from kitsu.io to kitsu.app.
  • Fixes 404 errors caused by Kitsu's recent domain and CDN switch.
  • Affected files: anilist.js, mal.js, anilist.ts, mal.ts, and docs.

What kind of change does this PR introduce?

This PR is a bugfix.

Did you add tests for your changes?

No new tests were added since this is a straightforward URL update. The existing tests should cover the relevant functionality.

If relevant, did you update the documentation?

Yes, the documentation has been updated to reflect the new Kitsu URLs.

Summary

This PR resolves an issue where Kitsu images were returning 404 errors due to their recent domain and CDN change from kitsu.io to kitsu.app. By updating the URLs, this ensures that all Kitsu-based features work smoothly without any disruption.

Other information

This update is essential for keeping everything running as expected. A quick review and merge would be appreciated to avoid further issues. Thanks!

- Updated Kitsu GraphQL and media URLs from `kitsu.io` to `kitsu.app`.
- Fixes `404` errors caused by Kitsu's recent domain and CDN switch.
- Affected files: `anilist.js`, `mal.js`, `anilist.ts`, `mal.ts`, and docs.
Copy link
Contributor

@Gratenes Gratenes left a comment

Choose a reason for hiding this comment

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

Simple domain changes, looks good to me. This merge just changes ".io" => ".app" for the kitsu provider.

@riimuru riimuru merged commit 9444dc7 into consumet:master Aug 14, 2024
4 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