-
Notifications
You must be signed in to change notification settings - Fork 212
[$50]Registration : Error on registering #4562
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
Contest https://www.topcoder.com/challenges/30130310 has been created for this ticket. |
Contest https://www.topcoder.com/challenges/30130310 has been updated - it has been assigned to luizrrodrigues. |
@sushilshinde PR's is ready:
@SathyaJayabal Before we go with QA we'll need API fix, check bellow: @rootelement When try to agree the term API return error |
@ThomasKranitsas please check this at the start of your day |
@SathyaJayabal please wait for API for fix |
The problem that @ThomasKranitsas fixed is that the terms api wasn't failing correctly, it was proceeding to the next middleware. See here: So while this will now throw an error when this call is made, we're not sure why that call should be failing. Can you give us steps to reproduce? You're calling with a JWT of what user? Is this in dev or prod? |
FYI - we're almost done debugging this. We found the root of the issue and have tested most of it. Will update shortly. |
@rootelement cool, thanks. Let me know if need more details. |
@luizrrodrigues Please try now. All our tests pass in dev. I will be pushing to Prod momentarily. |
@rootelement Working well now. Thanks. @sushilshinde @SathyaJayabal I'll need to do an additional fix in FE. |
@sushilshinde Hotfix is ready: |
#4562 luiz #4588 topcoder-platform/topcoder-react-lib#203 1000.19.28 #4603 topcoder-platform/topcoder-react-lib#204 1000.19.29
@luizrrodrigues @sushilshinde
|
@sushilshinde , @luizrrodrigues lets track the above issues in separate tickets |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30130310 |
#4380 (comment)
The text was updated successfully, but these errors were encountered: