[$50]Challenge Listing: Ongoing challenges Sort by Current phase does not work as expected

[SathyaJayabal]

Go to challenge listing
select Ongoing Challenges filter
select "Sort by Current Phase"

actual: the results are not sorted by the current phase
expected: the results must be sorted by current phase

Note: unable to add attachments. will upload later

[SathyaJayabal]

[SathyaJayabal]

@sushilshinde @Oanh-and-only-Oanh ,can we address this too with the v5 challenge-api release. The sorting is not working correctly in the v5 integration too. cc @lakshmiathreya

[Oanh-and-only-Oanh]

@SathyaJayabal, i think P2 is the right label for this one. Since we're not going to PROD until Aug. 3, we might be able to get this resolved with the first release. @sushilshinde

[crazyk07]

Contest https://www.topcoder.com/challenges/30137529 has been created for this ticket.

This is an automated message for crazyk via Topcoder X

[crazyk07]

Contest https://www.topcoder.com/challenges/30137529 has been updated - it has been assigned to cagdas001.

This is an automated message for crazyk via Topcoder X

[cagdas001]

PR: #4772

[sushilshinde]

Thanks, some technicical problem with the staging environemnts PR so not merged.

[lakshmiathreya]

Current Phase Alphabetical Sorting is working as expected in Staging.


Not working in V4 and Prod (V5):

[SathyaJayabal]

@Oanh-and-only-Oanh @sushilshinde @lakshmiathreya , what is the requirement for sort by current phase?

1. Is it alphabetical sort by phase name
   OR
2. Sort based on order in which phases are run. Example:
   a. Registration
   b. Checkpoint Submission
   c. Submission
   d. Checkpoint review
   e. Review/ Iterative Review
   f. Appeals
   g. Appeals Response
   h. Final Fix
   h Approval
   i. Post Mortem

[lakshmiathreya]

@SathyaJayabal the fix seems to be for Alphabetical order ...

[cagdas001]

@lakshmiathreya @SathyaJayabal Yes, the fix is Alphabetical order. The old implementation was using the Alphabetical order, so I followed the same thing.

[SathyaJayabal]

@cagdas001 , if the old implementation was using alphabetical order, then we are good.

[cagdas001]

@SathyaJayabal Yes, it was sorting in alphabetical order, can be seen here: https://github.com/topcoder-platform/community-app/pull/4772/files#diff-6c236f955a26f5eacaad6b5b079bdd63L24

Though, IMHO, this sorting might not be good for UX, but, since the old implementation was using this I followed the same.

You can discuss your proposal (I think it's better for UX) with team and if it's allowed I can change the sorting

Sort based on order in which phases are run. Example:
a. Registration
b. Checkpoint Submission
c. Submission
d. Checkpoint review
e. Review/ Iterative Review
f. Appeals
g. Appeals Response
h. Final Fix
h Approval
i. Post Mortem

[SathyaJayabal]

@cagdas001 , you are right about the UX. If there is a change, we will do it as an enhancement in a separate ticket.

[lakshmiathreya]

Alpha sort working fine in Prod.

[crazyk07]

Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30137529

This is an automated message for crazyk via Topcoder X