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

GetLeaderboardForSocialGroupAsync() fails for Favorite category #134

Closed
pnp2084 opened this issue Apr 24, 2017 · 3 comments
Closed

GetLeaderboardForSocialGroupAsync() fails for Favorite category #134

pnp2084 opened this issue Apr 24, 2017 · 3 comments

Comments

@pnp2084
Copy link

pnp2084 commented Apr 24, 2017

Calling GetLeaderboardForSocialGroupAsync() with the SocialGroupConstants::Favorite category fails because the Xbox Live SDK (UWP) code passes the text Favorite in the URL instead of favorite.

leaderboard_service::get_leaderboard_for_social_group_internal() should do something like:

 if (utils::str_icmp(socialGroup, _T("Favorite")) == 0)
 {
     group = _T("favorite");
 }
@adityatMS
Copy link
Member

Casing on the URL shouldn't matter. The correct fix is to change social_group_constants::favorite() to return _T("Favorite") instead of _T("Favorites"). I'll send a PR shortly with a fix.

@adityatMS
Copy link
Member

Sorry for the delay in getting back to you. We've decided to fix this on the service instead where the service will understand both 'Favorite' and 'Favorites'. I will update the thread when a fix is deployed. In the meantime, you can change the constant to 'Favorites' to unblock yourself.

@adityatMS
Copy link
Member

The fix for the issue was deployed yesterday. Please let us know if you are still seeing any issues.

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

3 participants