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

Unreliable data from API_GetUserRecentAchievements endpoint #2493

Open
ioslife opened this issue Jun 12, 2024 · 7 comments
Open

Unreliable data from API_GetUserRecentAchievements endpoint #2493

ioslife opened this issue Jun 12, 2024 · 7 comments
Labels
area/api-web Web API kind/bug Something isn't working

Comments

@ioslife
Copy link
Contributor

ioslife commented Jun 12, 2024

Hello, one of my users pointed out that the Recent Achievements tab for them wasn't showing the most up to date data. I pulled his data from the API and it isn't showing anything past 6/7 even though he has earned cheevos since.

User: UltraGodAzgorath
Endpoint: API_GetUserRecentAchievements

Pics: 1st is RA.org progress page. 2nd is Postman API response

image image
@wescopeland wescopeland added kind/bug Something isn't working area/api-web Web API labels Jun 12, 2024
@ioslife
Copy link
Contributor Author

ioslife commented Jul 1, 2024

I have another report from another user with the same issue: Fngreg7

@Danielopes7
Copy link

Danielopes7 commented Jul 14, 2024

@wescopeland Can I implement pagination on this API? I think this solve the problem.

@wescopeland
Copy link
Member

Sorry, I haven't investigated this issue and I'm not quite sure what the root cause of the problem is. Assuming I'm understanding the question correctly, I'm not sure why pagination might necessarily solve the issue if we're currently returning all the records.

@Danielopes7
Copy link

Sorry, I haven't investigated this issue and I'm not quite sure what the root cause of the problem is. Assuming I'm understanding the question correctly, I'm not sure why pagination might necessarily solve the issue if we're currently returning all the records.

in the query has the limit of 500

@wescopeland
Copy link
Member

Ah, I see. Thank you for that explanation. You're free to proceed fixing this. My requirements would be as follows:

  • If we're going to manipulate the query, we should migrate it from raw SQL to Eloquent ORM.
  • The pagination parameters should be modeled after our other collection responses, this PR is a good example.
  • We'll need comprehensive test coverage of the new parameters.
  • We'll need an accompanying docs PR like this one.

@wescopeland
Copy link
Member

Note: If the pagination parameters are not used, we'll need to preserve the exact response shape we currently have so we don't introduce a breaking change.

@Danielopes7
Copy link

Note: If the pagination parameters are not used, we'll need to preserve the exact response shape we currently have so we don't introduce a breaking change.

I was going to ask about that, thanks for clarifying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api-web Web API kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants