Skip to content
This repository has been archived by the owner on Jan 14, 2018. It is now read-only.

Some changes / suggest #5

Open
grambas opened this issue Jun 16, 2014 · 0 comments
Open

Some changes / suggest #5

grambas opened this issue Jun 16, 2014 · 0 comments

Comments

@grambas
Copy link

grambas commented Jun 16, 2014

Hello again.

Riot API has changed url for euw servers
from riot page:
New endpoint for EUW APIs. The EUW endpoint has changed from prod.api.pvp.net to euw.api.pvp.net. The prod.api.pvp.net will no longer work to access EUW services.

I Don't know if its issue for your code, but if yes, please change it.

Im still working with your API. Big thanks that you have write it. But I lack some features.
I am making registration for lol players. Can you provide best way to first check if its returns data and only then execute url? Becouse some times Summoner doesnt have ranked stats and when I pulling it am getting errors. Also on mistake bu writing summoner name. Now I using :
$api = new RiotApi($region);
$summonerByName = $api->getSummonerByName($ign);
if ($summonerByName == NULL) {
do action
}
should i do it on all request?
Best Regards, Mindaugas

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant