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

LeagueListDTO does not match returned JSON #29

Closed
stelar7 opened this issue Dec 10, 2018 · 9 comments
Closed

LeagueListDTO does not match returned JSON #29

stelar7 opened this issue Dec 10, 2018 · 9 comments
Labels
api: league-v4 closed: resolved topic: documentation type: inconsistency Typos, dead links, out dated information, or weird behavior.

Comments

@stelar7
Copy link

stelar7 commented Dec 10, 2018

The fields playerOrTeamId and playerOrTeamName has been renamed in v4.
There might be some other changes too, but they are all undocumented, so i dont know.

@RiotTuxedo
Copy link
Contributor

Do you have an endpoint and sample response where these fields aren't what you're expecting?

{
    "tier": "MASTER",
    "leagueId": "bb7f45c3-66d1-3b22-bc68-7a6c65b38c4e",
    "entries": [
        {
            "wins": 665,
            "losses": 636,
            "rank": "I",
            "playerOrTeamName": "Lycho",
            "playerOrTeamId": "44472302",
            "leaguePoints": 0
        }
    ]
}

@RiotTuxedo RiotTuxedo added on hold: clarification needed This issue is unclear and requires additional information. topic: riot games api labels Dec 10, 2018
@Derpthemeus
Copy link

https://na1.api.riotgames.com/lol/league/v4/positions/by-summoner/rtBoABIL8T_J690zXoVKNSJDTJ20K6Db5d59RjCXMudF9P4

[
    {
        "queueType": "RANKED_SOLO_5x5",
        "summonerName": "Derpthemeus",
        "wins": 35,
        "losses": 38,
        "rank": "IV",
        "leagueName": "Karma's Commandos",
        "leagueId": "926dd230-2340-11e8-bb3d-c81f66cf135e",
        "tier": "GOLD",
        "summonerId": "rtBoABIL8T_J690zXoVKNSJDTJ20K6Db5d59RjCXMudF9P4",
        "leaguePoints": 0
    },
    {
        "queueType": "RANKED_FLEX_SR",
        "summonerName": "Derpthemeus",
        "wins": 17,
        "losses": 17,
        "rank": "IV",
        "leagueName": "Veigar's Berserkers",
        "leagueId": "0e3a89e0-69b3-11e8-a566-c81f66dbb56c",
        "tier": "GOLD",
        "summonerId": "rtBoABIL8T_J690zXoVKNSJDTJ20K6Db5d59RjCXMudF9P4",
        "leaguePoints": 0
    }
]

@RiotTuxedo RiotTuxedo added pending: acknowledged We're aware of this issue and are investigating. and removed on hold: clarification needed This issue is unclear and requires additional information. labels Dec 10, 2018
@MingweiSamuel
Copy link

To clarify, it seems that LeagueItemDTO's documentation still contains the old "playerOrTeamName" and "playerOrTeamId" when in reality it is returning the new "summonerName" and "summonerId".

This affects the league endpoints: (since they use LeagueListDTO which uses LeagueItemDTO)

  • /lol/league/v4/challengerleagues/by-queue/{queue}
  • /lol/league/v4/grandmasterleagues/by-queue/{queue}
  • /lol/league/v4/leagues/{leagueId}
  • /lol/league/v4/masterleagues/by-queue/{queue}

Positions is not affected because it uses Set[LeaguePositionDTO] instead:

  • /lol/league/v4/positions/by-summoner/{encryptedSummonerId}

@MingweiSamuel
Copy link

MingweiSamuel commented Dec 15, 2018

The following fields are removed/missing (from the API response)

name type
hotStreak boolean
veteran boolean
freshBlood boolean
(playerOrTeamName) string
inactive boolean
(playerOrTeamId) string

@MingweiSamuel
Copy link

MingweiSamuel commented Dec 16, 2018

The following fields are removed/missing from LeagueListDTO (in the API response)

NAME DATA TYPE
queue string
name string

@kargnas
Copy link

kargnas commented Jan 13, 2019

The attributes hotStreak, veteran, freshBlood, inactive should be available on v4 LeagueListDTO, but still I can't see these attributes.

@RiotTuxedo
Copy link
Contributor

I've confirmed these missing fields are the result of a bug and they should return early this next week.

@gustavoja
Copy link

Hello,
Looking at this conversation and the changes posted here

I think you may need to update the LeaguePostionDTO doc because you're still referencing the field "playerOrTeamName string" which has changed to summonerName.

captura de pantalla 2019-01-18 a las 11 19 32

Thank you

@MingweiSamuel
Copy link

(Fixed, this issue can be closed)

@RiotTuxedo RiotTuxedo added closed: resolved topic: documentation type: inconsistency Typos, dead links, out dated information, or weird behavior. and removed pending: acknowledged We're aware of this issue and are investigating. topic: riot games api type: bug report labels Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: league-v4 closed: resolved topic: documentation type: inconsistency Typos, dead links, out dated information, or weird behavior.
Projects
None yet
Development

No branches or pull requests

6 participants