-
Notifications
You must be signed in to change notification settings - Fork 212
[$50]Sort by: Time to submit and Time to register does not work as expected #4715
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
Contest https://www.topcoder.com/challenges/30136118 has been created for this ticket. |
@topcoder-platform/topcodercompetitors @topcoder-platform/tcxcommunity this is open for pickup Submit in 8 hours and PR to https://github.com/topcoder-platform/community-app/tree/integration-v5-challenge-api run |
Contest https://www.topcoder.com/challenges/30136118 has been updated - it has been assigned to Narekvar90. |
Contest https://www.topcoder.com/challenges/30136118 has been updated - it has been assigned to cagdas001. |
Fix/rewrite TIME_TO_REGISTER and TIME_TO_SUBMIT sorting functions. Addresses topcoder-platform#4715
PR: #4725 It will first sort the upcoming challenges (i.e. you have still time to submit/register), by ascending order. Then sort the past challenges (i.e. submission/registration phase is in past) in the descending order. For e.g., you'll see a sorting like this |
Thanks, some technicical problem with the staging environemnts PR so not merged. |
@cagdas001 Can you fix tests? |
@sushilshinde done |
@SathyaJayabal pls review. My Challenges Time to Submit sort listing is the same in Staging and Prod. |
@cagdas001, for time to register sort (Open for registration bucket), the late challenges must be displayed first. But the late challenges are displayed in the end. |
@SathyaJayabal Yes, because there is no possibility to register on late challenges, phase is already closed. So I thought it'd be better for user if we first show the challenges that has still time for registration/submission. The current sorting is as follows:
Do we need to show late challenges first? |
@cagdas001, You are right |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30136118 |
The following sort functionalities on the challenge listings page does not work as expected.
My Challenges bucket: Time to Submit
Open for registration bucket: Time to Submit, Time to Register
The text was updated successfully, but these errors were encountered: