Skip to content

[$55] Past Challenges Search : Search takes a long time to display results if no challenges were found #3610

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.

Sign up for GitHub

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

Jump to bottom
Closed
SathyaJayabal opened this issue Jan 24, 2020 · 13 comments
Closed

[$55] Past Challenges Search : Search takes a long time to display results if no challenges were found #3610

SathyaJayabal opened this issue Jan 24, 2020 · 13 comments
Assignees
@gets0ul
Labels
Challenge Listing P4 Low (resolve within 2 weeks) Prod Env Environment QA Pass tcx_Assigned tcx_FixAccepted tcx_Paid
Milestone
v0.20.01

Comments

@SathyaJayabal
Copy link
Collaborator

SathyaJayabal commented Jan 24, 2020 •
edited by LieutenantRoger
Loading

  1. Go to Past Challenges
  2. Search for a keyword that is does not return any results

Expected: The search results (Message "No Past Challenges found") must be displayed at a reasonable amount of time.

Actual: The search keeps going on for more than a few minutes.

Please see video
https://drive.google.com/open?id=1jwq43QZpOgPzeR1lv-1b0oi_ErqeBOax

Please make the text style look like this: https://take.ms/HCDrt (the rectangle part and the white background color can be ignored)

The work should be done against hot-fix-past-challenge-search
@SathyaJayabal SathyaJayabal added Challenge Listing P4 Low (resolve within 2 weeks) Prod Env Environment labels Jan 24, 2020
@Oanh-and-only-Oanh Oanh-and-only-Oanh mentioned this issue Jan 24, 2020
Closed
@Oanh-and-only-Oanh Oanh-and-only-Oanh removed their assignment Jan 24, 2020
@LieutenantRoger LieutenantRoger changed the title Past Challenges Search : Search takes a long time to display results if no challenges were found [$35] Past Challenges Search : Search takes a long time to display results if no challenges were found Jan 24, 2020
@LieutenantRoger
Copy link
Collaborator

Hey @topcoder-platform/topcodercompetitors

This is open for pickup.

@crazyk07
Copy link

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

This is an automated message for crazyk via Topcoder X

@LieutenantRoger LieutenantRoger changed the title [$35] Past Challenges Search : Search takes a long time to display results if no challenges were found [$45] Past Challenges Search : Search takes a long time to display results if no challenges were found Jan 27, 2020
@LieutenantRoger
Copy link
Collaborator

Hi @topcoder-platform/topcodercompetitors

We increased the price for the ticket, please come and pick it up.

Thanks.

@LieutenantRoger LieutenantRoger changed the title [$45] Past Challenges Search : Search takes a long time to display results if no challenges were found [$55] Past Challenges Search : Search takes a long time to display results if no challenges were found Jan 29, 2020
@LieutenantRoger
Copy link
Collaborator

Hi @topcoder-platform/topcodercompetitors

We increased the price for the ticket again, come and help us resolve this issue.

Thanks.

@gets0ul gets0ul self-assigned this Jan 29, 2020
@crazyk07
Copy link

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

This is an automated message for crazyk via Topcoder X

@gets0ul
Copy link
Collaborator

gets0ul commented Jan 29, 2020

@LieutenantRoger on which branch? develop?

@Oanh-and-only-Oanh Oanh-and-only-Oanh removed the Prod Env Environment label Jan 29, 2020
@LieutenantRoger
Copy link
Collaborator

@gets0ul , sorry for not providing this, please work against this branch hot-fix-past-challenge-search

@gets0ul gets0ul mentioned this issue Jan 30, 2020
Merged
LieutenantRoger added a commit that referenced this issue Jan 31, 2020
@LieutenantRoger
Merge pull request #3626 from gets0ul/issue_3610
fd5670a 
Fix Issue #3610 Waiting for Past Challenge Search with no result.
@SathyaJayabal
Copy link
Collaborator Author

@LieutenantRoger , search works fine. Results displayed in 35 seconds.
Can we update the text to "No challenges found in Past Challenges" to keep in line with the other filters.

Screenshot 2020-01-31 at 10 46 49 AM

@SathyaJayabal SathyaJayabal added Dev Env QA Fail QA verification on Dev has failed. Assignee to redo the fix. and removed QA Fail QA verification on Dev has failed. Assignee to redo the fix. labels Jan 31, 2020
@SathyaJayabal
Copy link
Collaborator Author

Verified in dev. Result displayed in 35 seconds
Screenshot 2020-01-31 at 12 30 44 PM

@LieutenantRoger
Copy link
Collaborator

LieutenantRoger commented Jan 31, 2020 •
edited
Loading

Thanks. FYI, the current configured time out time is 30 seconds.

@LieutenantRoger LieutenantRoger mentioned this issue Jan 31, 2020
Merged
@SathyaJayabal SathyaJayabal mentioned this issue Feb 4, 2020
Closed
@SathyaJayabal
Copy link
Collaborator Author

Verified in Beta. Displayed results after 30 seconds.
Screenshot 2020-02-11 at 1 25 12 PM

@SathyaJayabal SathyaJayabal added Beta Env Environment and removed Dev Env labels Feb 11, 2020
@LieutenantRoger LieutenantRoger mentioned this issue Feb 11, 2020
Merged
@SathyaJayabal
Copy link
Collaborator Author

verified in production. Result displayed after 30 secs.
Screenshot 2020-02-11 at 3 06 48 PM

@SathyaJayabal SathyaJayabal added Prod Env Environment and removed Beta Env Environment labels Feb 11, 2020
@crazyk07
Copy link

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

This is an automated message for crazyk via Topcoder X

@crazyk07 crazyk07 added Beta Env Environment tcx_Paid and removed Prod Env Environment labels Feb 11, 2020
@SathyaJayabal SathyaJayabal added this to the 0.20.01 milestone Feb 12, 2020
@SathyaJayabal SathyaJayabal added Prod Env Environment and removed Beta Env Environment labels Feb 13, 2020
@LieutenantRoger LieutenantRoger mentioned this issue Mar 14, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gets0ul gets0ul

Labels
Challenge Listing P4 Low (resolve within 2 weeks) Prod Env Environment QA Pass tcx_Assigned tcx_FixAccepted tcx_Paid
Projects
None yet
Milestone
v0.20.01
Development

No branches or pull requests
5 participants
@LieutenantRoger @gets0ul @crazyk07 @Oanh-and-only-Oanh @SathyaJayabal