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

GetPerson request failed #33

Closed
Arzack010 opened this issue Dec 27, 2021 · 5 comments
Closed

GetPerson request failed #33

Arzack010 opened this issue Dec 27, 2021 · 5 comments

Comments

@Arzack010
Copy link

I think its because myanimelist.net change person to people in url
https://myanimelist.net/person/{id} return an error
the correct url that return a data is:
https://myanimelist.net/people/{id}

@Ervie
Copy link
Owner

Ervie commented Dec 27, 2021

Hi, if that was a case, the issue would be related to main Jikan Api.

However, I tested few calls from library and they were successful. Can you post an example of the person id that failed for you? From top of my head, there's a list for possible causes:

  • There's no person under this id (it happens sometimes on MAL)
  • You are being rate limited and need to lessen amount of calls (see documentation of the API)
  • Internal MAL issues, e. g. site is down

But would be best to work on example. I also suggest to check the exception message and code in thrown exception.

@Arzack010
Copy link
Author

I try with the same example from the wiki
example

according of the wiki the method GetPerson parses item data by ID from https://myanimelist.net/person/{id}
but if you go to https://myanimelist.net/person/1870 you get a "404 Not Found"

@Ervie
Copy link
Owner

Ervie commented Dec 28, 2021

It's working for some ids (lik 1 or 7), but failing for few different. I created a an issue on official API repo, we'll see how they respond. If it's fixed on API side, then this wrapper should work automatically.

@Ervie
Copy link
Owner

Ervie commented Dec 28, 2021

He already fixed that. You can check it on your side.

@Arzack010
Copy link
Author

Thanks, it works

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

No branches or pull requests

2 participants