RiotGames / developer-relations

Riot Games Developer Ecosystem Bug Reporting
http://developer.riotgames.com
743 stars 44 forks source link

[FEATURE REQUEST] return TFT b-patches as part of tft_match info response body #820

Open orlandosh opened 1 year ago

orlandosh commented 1 year ago

Problem Description
When I request a match by its ID, game_version returns "Version 13.18.530.4653 (Sep 07 2023/19:32:41) [PUBLIC]". However, the current tft version is not from Sep 07, it's from Sep 15th.

Expected Result
I was expecting the version to correctly indicate the date, Sep 15th, so my app can detect it's a new b-patch.

Actual Result
Version 13.18.530.4653 (Sep 07 2023/19:32:41) [PUBLIC]

Developer Impact
Describe how this affects you as a developer. Please include the frequency and severity of the issue.

Player Impact
Describe how this affects League of Legends players. Please include the frequency and severity of the issue.

Preconditions
What conditions must be met in order for this bug to occur?

Repro Steps
Step by step, what do you need to do to cause the bug? Note that GitHub will automagically number each step if you prefix the line with "1. "

  1. query riot tft_match after a b patch is live

Issue Comments
This is an unexpected issue. I actually created a system to detect when a b-patch is live through the api response but I say that the responses haven't changed, so the game version field ends up not being useful for b-patches. I will probably have to contact riot api support to find a compliant way to detect b patches automatically, as doing it manually is not viable either. I haven't explored the in-game API as an standalone app is still not part of my roadmap.

bangingheads commented 1 year ago

This is the correct game client version as noted in the documentation of the field.

This would be more of a feature request to include this data than a bug.

orlandosh commented 1 year ago

This is the correct game client version as noted in the documentation of the field.

This would be more of a feature request to include this data than a bug.

Thanks for pointing it out, updated accordingly.