-
Notifications
You must be signed in to change notification settings - Fork 212
[$60]Challenge Details: Eligible events section not displayed #4430
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
FYI: This issue was tested on production backend. Please use the given urls to compare |
Contest https://www.topcoder.com/challenges/30126891 has been created for this ticket. |
Contest https://www.topcoder.com/challenges/30126891 has been updated - it has been assigned to sr_jr. |
@sushilshinde @rootelement can you help how do we get |
This has to be migrated. Working on adding this to the metadata object. it will be as such: "metadata": [
{
"events": [
{
"id": 3446,
"name": "2016 TopCoder(R) Open",
"key": "tco16"
}]
}
], *Edited to make the structure an array, as expected |
@rootelement , I think there is a link also associated with the text |
Not according to the v4 response |
it gets id, event_desc, and event_short_desc |
@rootelement , I see a link also here in this example. |
That link is built on the frontend, it's not part of the api response |
This is done in dev: Will run on prod overnight |
@simranb86 waiting on PR |
Sorry I thought I already pushed it, has it ready once received the info. Pushed in PRs now. |
@sushilshinde , can see the eligible events on dev, but not on prod |
@SathyaJayabal yes, this is data issue, plz make sure you test with challenge which has events in it - try this via postman |
@simranb86, on testing after the data migration in dev. I can see the events in the api response, but not in the front end. Can you please take a look. |
@SathyaJayabal The backend impl was changed, it is re-work to be done here. The metadata was an object but now it has been updated to be an array. Now I get this
@rootelement can you confirm whether metadata will be array or object as it would mean re-work again, if I change it to be an array now and in future we return object again ? @sushilshinde FYI ☝️ |
I apologize @simranb86 , metadata was always intended to be an array of objects with {type, value}. See here: Events is no different: |
Thanks @rootelement @SathyaJayabal @sushilshinde give me like 30 mins, will raise new PR for the changes. Thanks |
@simranb86 please link PR |
@sushilshinde this is pending for @rootelement to provide a challenge which has events in valid format. I already flagged him on slack and waiting for him to provide one which could be used for development and then can pass to qa to validate it. |
Events is now a first class object: Also, metadata is please update accordingly and see my big update in slack for today. |
Thanks @rootelement @sushilshinde PR for the same topcoder-platform/topcoder-react-lib#180 |
Thanks @simranb86 |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30126891 |
V5 URL: https://beta-community-app.topcoder.com/challenges/d9c74629-762f-4464-9753-ebd982c7990f
V4 URL: https://www.topcoder.com/challenges/30106190


The text was updated successfully, but these errors were encountered: