topcoder-platform / work-manager

This is the frontend application for creating and managing challenges.
13 stars 48 forks source link

The email addresses of some members are not displayed in WM Registrants and Submissions tabs #1545

Closed jmgasper closed 1 year ago

jmgasper commented 1 year ago

REF: https://topcoder.atlassian.net/browse/PROD-4296

Steps

891be3bf-cc6a-48a7-b0dc-be07c278f1ba#media-blob-url=true id=f81b1178-f53e-443c-b9ad-6d7746243bb6 collection= contextId=39731 height=786 width=1523 alt=

Diagnosis

The problem here is that if there are more than 50 registrants, the call to the member API is paginated, so only the first 50 are returned, and we aren't pulling subsequent pages. Can we make asynchronous calls to get more member emails after the page loads and just fill them in as the response is returned?

jmgasper commented 1 year ago

@suppermancool - I think we need to paginate without sending all user IDs. I tested a challenge with 400 registrants, and we get a response of 414 - Request URI Too Large because all the member IDs are in the original URL.

jmgasper commented 1 year ago

Challenge: https://challenges.topcoder-dev.com/projects/14096/challenges/f0e33ae2-336d-413e-856b-d9d35848139e/view

jmgasper commented 1 year ago

@suppermancool - I'm still not seeing this work quite right. Can you test with mess / appirio123 and https://challenges.topcoder-dev.com/projects/14096/challenges/f0e33ae2-336d-413e-856b-d9d35848139e/view please?

suppermancool commented 1 year ago

@jmgasper Actually the api for fetching member email by member id only allows 50 members. done in work-manager-issue-1545.patch