-
Notifications
You must be signed in to change notification settings - Fork 212
Make the stats history data to support groupIds query params and use new API to render the stats history UI #4137
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
Comments
Also please only test |
@LieutenantRoger , was able to view both the public and private stats history page for the user standlove. There are a few inconsistencies in the data like the below
|
As have discussed, this is the expected behavior due to the testing data preparation. Btw, I have re-deployed the feature using topcoder-react-lib#v1000.14.0 to dev env for QA. |
@LieutenantRoger for the. topcoder stats history page(public) there should not abe any group id passed right? |
@LieutenantRoger for the call to stats history page for topgear, there are. two group ids passed. Topgear group is "20000000", why is the other id passed ? |
It's expected as well, please refer the |
Correct, when there is no group id provided in the request, the history data with groupId=10 will be returned by default. |
@LieutenantRoger , I am. passing this one as per clarifications above. The public and topgear stats history pages are displayed for the user standlove. The group id is passed in the history call. |
Thanks @SathyaJayabal |
@lakshmiathreya please close if the ticket is resolved. It's been long time now. |
@LieutenantRoger ,the stats data is still not visible for top gear users in production. @lakshmiathreya can u verify this please. Clicking on any user from the topgear leaderboard should show the profile page with stats. |
@LieutenantRoger the below user must have stats data in topgear but the stats are not showing up |
@SathyaJayabal got it. I just remembered the production data under topgear are still under processing, I think this is the reason that no data is returned, we can keep the ticket as open now, thanks. |
@LieutenantRoger need pr for this |
marking this as hold until backend data is ready for topgear stats history on prod |
@Oanh-and-only-Oanh I am reopening this - it appears like Code was fixed but backend data was not available. Will leave this open until we see Stats data fully. cc @urwithat @SathyaJayabal |
https://github.com/topcoder-platform/community-app/tree/feature-stats-history
Please make sure both public history page and topgear history page are working correctly.
https://github.com/topcoder-platform/community-app/tree/feature-stats-history
https://github.com/topcoder-platform/topcoder-react-lib/tree/feature-stats-history
The text was updated successfully, but these errors were encountered: