Skip to content

[$30]Challenge Listings: Sometimes the challenges do not load and the animation placeholder is displayed for a long time #4491

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

Closed
SathyaJayabal opened this issue Jun 8, 2020 · 19 comments

Comments

@SathyaJayabal
Copy link
Collaborator

https://test-community-app.topcoder-dev.com/challenges

Sometimes the challenges do not load and the animation placeholder is displayed for a long time.

Please see attached video.
perf.mov.zip

@sushilshinde sushilshinde changed the title Challenge Listings: Sometimes the challenges do not load and the animation placeholder is displayed for a long time [$30]Challenge Listings: Sometimes the challenges do not load and the animation placeholder is displayed for a long time Jun 8, 2020
@crazyk07
Copy link

crazyk07 commented Jun 8, 2020

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

This is an automated message for crazyk via Topcoder X

@crazyk07
Copy link

crazyk07 commented Jun 8, 2020

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

This is an automated message for crazyk via Topcoder X

@srjr19
Copy link
Contributor

srjr19 commented Jun 9, 2020

@SathyaJayabal @sushilshinde what is load time we looking at since we have lot of backend calls going on, so that maybe causing this issue I guess.

@SathyaJayabal
Copy link
Collaborator Author

@simranb86 , Sometimes the page loads within 2 seconds, other times takes more than 3 or 4 minutes. I am not sure if this is due to performance issue or something else.

@srjr19
Copy link
Contributor

srjr19 commented Jun 9, 2020

@simranb86 , Sometimes the page loads within 2 seconds, other times takes more than 3 or 4 minutes. I am not sure if this is due to performance issue or something else.

Ok I have changed the logic a bit, pushing a PR in few mins for #4383 which should speed this up as number of calls have been brought down.

@srjr19
Copy link
Contributor

srjr19 commented Jun 9, 2020

@sushilshinde this should also speed this page up, #4383 (comment) - so moving to review so we can test once the changes are merged and will address if we still see issues.

@SathyaJayabal SathyaJayabal added QA Fail QA verification on Dev has failed. Assignee to redo the fix. and removed Ready for QA labels Jun 11, 2020
@SathyaJayabal
Copy link
Collaborator Author

@simranb86 @sushilshinde , I still see this issue.
perf.mov.zip

@SathyaJayabal
Copy link
Collaborator Author

Another example on beta.
perf_beta.mov.zip

@srjr19
Copy link
Contributor

srjr19 commented Jun 11, 2020

@SathyaJayabal not able to replicate - see here https://youtu.be/kZpljrXy2PQ - I try several times in this video. Can you hold on and show the full flow as you say it loads, so what is time it takes to load and can you also enable your console logs while recording so we can see what is going on ?

@SathyaJayabal
Copy link
Collaborator Author

@simranb86 , here is the console logs.
Screenshot 2020-06-12 at 9 57 30 AM
I will upload the video shortly

@SathyaJayabal
Copy link
Collaborator Author

@srjr19
Copy link
Contributor

srjr19 commented Jun 12, 2020

@simranb86 , here is the video
https://drive.google.com/file/d/1zEkevRrbsHOuSADv9ucK5cIzLPcu0dNo/view?usp=sharing

Don't have access to it, can you push to YouTube maybe or share shareable link ?

@SathyaJayabal
Copy link
Collaborator Author

SathyaJayabal commented Jun 12, 2020

@simranb86 gave you access. I have attached the video here too.
4491.mp4.zip

@srjr19 srjr19 removed the QA Fail QA verification on Dev has failed. Assignee to redo the fix. label Jun 12, 2020
@srjr19
Copy link
Contributor

srjr19 commented Jun 15, 2020

@sushilshinde based on logs, I am assuming this is causing the issue, haven't been able to reproduce locally. PR is here topcoder-platform/topcoder-react-lib#192

sushilshinde added a commit to topcoder-platform/topcoder-react-lib that referenced this issue Jun 15, 2020
sushilshinde added a commit that referenced this issue Jun 15, 2020
@SathyaJayabal
Copy link
Collaborator Author

SathyaJayabal commented Jun 15, 2020

@sushilshinde @simranb86 , we still see this issue
Test env - Logged in user
Screenshot 2020-06-15 at 8 58 12 PM
Beta Env - Logged in user
Screenshot 2020-06-15 at 8 58 14 PM
Test Env- Not logged in user
Screenshot 2020-06-15 at 9 00 28 PM
I could not replicate the issue for beta env for not logged in user.

@SathyaJayabal SathyaJayabal added QA Fail QA verification on Dev has failed. Assignee to redo the fix. and removed Ready for QA labels Jun 15, 2020
@srjr19
Copy link
Contributor

srjr19 commented Jun 16, 2020

@sushilshinde opening since not sure what else could be causing this.

@srjr19 srjr19 removed their assignment Jun 16, 2020
@Oanh-and-only-Oanh Oanh-and-only-Oanh removed QA Fail QA verification on Dev has failed. Assignee to redo the fix. P1 Urgent (resolve within 24 hours) labels Jun 16, 2020
@sushilshinde
Copy link
Collaborator

@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
https://github.com/topcoder-platform/topcoder-react-lib/tree/integration-v5-challenge-api

run npm run test before PR

@Oanh-and-only-Oanh Oanh-and-only-Oanh added the P2 Important (resolve within 3 days) label Jun 23, 2020
@sushilshinde sushilshinde added P4 Low (resolve within 2 weeks) and removed P4 Low (resolve within 2 weeks) P2 Important (resolve within 3 days) labels Jun 24, 2020
@SathyaJayabal
Copy link
Collaborator Author

@sushilshinde @lakshmiathreya This issue has not been observed recently. Closing it for now.

@crazyk07
Copy link

crazyk07 commented Jul 3, 2020

This ticket was not processed for payment. If you would like to process it for payment, please reopen it, add the tcx_FixAccepted label, and then close it again

This is an automated message for crazyk via Topcoder X

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants