-
Notifications
You must be signed in to change notification settings - Fork 212
[$30]Dashboard SRMS pick timings from Contest Table, Should pick up from Round Table - Registration Date #1867
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
@ajefts Who should I assign this issue to? |
@harshitmehta123 I'm checking with @skyhit on this, probably an API issue. |
@sushilshinde @harshitmehta123 seems not API, but UI, the response is like
|
Currently, it is picking Contest: startTime ....but we need to change it to registrationStartAt time in the rounds array |
@harshitmehta123 does registrationStartA has the correct value in this response? |
@topcoder-platform/topcodercompetitors this one is open for pickup PR on https://github.com/topcoder-platform/community-app/tree/hot-fixes |
Contest https://www.topcoder.com/challenges/30079108 has been created for this ticket. |
Contest https://www.topcoder.com/challenges/30079108 has been updated - it has been assigned to gurmeetb. |
Contest https://www.topcoder.com/challenges/30079108 has been updated - the new changes has been updated for this ticket. |
@sushilshinde looks like this is working fine already. Kindly validate once on your side on PROD. |
@harshitmehta123 please comment |
@sushilshinde I am putting this on hold until we hear back from @harshitmehta123 |
Hey! @sushilshinde @gurmeetb |
@harshitmehta123 can you verify |
Hi! Thanks for fixing! |
Should it not display the current time zone, like the arena does, or if not - it should have time zone written alongside it :)
|
Ok |
@harshitmehta123 shall we close this |
Thanks for adding the time zone, however - the time displayed is UTC and the time zone it's showing is UTC+5:30. Registration for SRM 748 begins at 22:30 IST on Jan 25. The time shown in the screenshot above it 3:30am which is UTC time .. but the time zone mentioned below is UTC+5:30. |
@sushilshinde @harshitmehta123 any updates here☝️ |
Hi @gurmeetb, I think there is an issue with the timings the API is returning.. These are the SRM timings in various time zones. I am investigating and will as soon as possible. Round Details: SRM 749 — February 1, 7:00 UTC-5 |
Shall we close this issue now Timezone text and other things are fixed by @gurmeetb? @harshitmehta123 you can open an issue on API repo. |
I agree with @sushilshinde - once API is fixed, this will work fine as it does with current time coming in response. @harshitmehta123 thoughts 🤔 ? |
@harshitmehta123 please create ticket on https://github.com/appirio-tech/ap-challenge-microservice/ |
This ticket was not processed for payment. If you would like to process it for payment, please reopen it, add the |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30079108 |
Describe the bug
SRMs displayed on dashboard pick date from Contest Table in OLTP. However, they should be picking the date from Round table one single contest can have multiple rounds
To Reproduce
Go to topcoder.com/my-dashboard and look for SRMs.
Looking at the screenshot you can see the SRMs picking contest date, however they should pick the round registration date marked in green :)
The text was updated successfully, but these errors were encountered: