Skip to content

[$30] review type needs auth token #4505

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
sushilshinde opened this issue Jun 9, 2020 · 7 comments
Closed

[$30] review type needs auth token #4505

sushilshinde opened this issue Jun 9, 2020 · 7 comments

Comments

@sushilshinde
Copy link
Collaborator

  1. Hit this url https://test-community-app.topcoder-dev.com/challenges
  2. Check newtwork console
  3. This call doesn't have token https://api.topcoder-dev.com/v5/reviewTypes

Fix: add Auth token to the call

@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

@crazyk07
Copy link

crazyk07 commented Jun 9, 2020

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

This is an automated message for crazyk via Topcoder X

@crazyk07
Copy link

crazyk07 commented Jun 9, 2020

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

This is an automated message for crazyk via Topcoder X

@sushilshinde sushilshinde added the P1 Urgent (resolve within 24 hours) label Jun 9, 2020
@PrakashDurlabhji
Copy link
Contributor

@sushilshinde

  1. is latest topcoder-react-lib used?
  2. also this reviewTypes api works fine and has auth token when user is logged in.

so do you want that without authentication as well if someone visits challenge-detail page then it should work? because I investigated and it works fine if user is logged in.

@sushilshinde
Copy link
Collaborator Author

sushilshinde commented Jun 11, 2020

@PrakashDurlabhji can we make sure, there is no call if the user is not logged in?

@PrakashDurlabhji
Copy link
Contributor

@sushilshinde I hope topcoder-react-lib is integrated perfectly and no different version because code seems ok at topcoder-react-lib, update you again in sometime

@crazyk07
Copy link

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

@lakshmiathreya
Copy link

@sushilshinde could you pls doublecheck and mark label "CF Retested" ...

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

4 participants