[$45] Able to submit even without registering

[LieutenantRoger]

Summary:
Able to upload Submission for a challenge even without registering

Steps to Reproduce:

1. Navigate to Some Challenge and Register.
2. Right Click on Submit and Open in New Tab
3. Now go back to the Challenge Details page and Unregister
4. Now go to the other tab where Submission page is open
5. Check if you are able to make the submission

Actual Result:
User is able to make submission even without registering

Expected Result:
User should not be let to make the submission since user is unregistered from the Challenge

Environment Details:
Devices(s):
PC
Operating System:
Windows 10
Browser and Version:
Chrome: 94.0.4606.81

[LieutenantRoger]

Challenge https://www.topcoder.com/challenges/6a184130-5370-4cea-98ed-96829b22c969 has been created for this ticket.

This is an automated message for lieutenantroger via Topcoder X

[fikzzzy]

@bug-bash-helper assign

[bug-bash-helper]

@fikzzzy ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

[bug-bash-hunt-helper]

@fikzzzy ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

[fikzzzy]

@bug-bash-helper unassign

[bug-bash-helper]

@fikzzzy ✅ you have been unassigned from this issue.

Now you may pick up another issue which is open for pickup if you like to.

[bug-bash-hunt-helper]

@fikzzzy ✅ you have been unassigned from this issue.

Now you may pick up another issue which is open for pickup if you like to.

[ben-mathu]

@bug-bash-helper assign

[bug-bash-helper]

@ben-mathu ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

[bug-bash-hunt-helper]

@ben-mathu ✅ you are now assigned to this issue and have 12 hours to complete it.

As soon as you are done, please, make a comment like below, including the link to the pull request:

@bug-bash-helper <link to PR> is ready for review

[dprincecoder]

@bug-bash-helper assign

[bug-bash-helper]

@dprincecoder 🛑 this issue is not open for pick up.

You may only pickup issues which are included in this Bug Bash and open for pick up.
Such issues have open status and have labels CF and Open for Pickup.

[bug-bash-hunt-helper]

@dprincecoder 🛑 this issue is not open for pick up.

You may only pickup issues which are included in this Bug Bash and open for pick up.
Such issues have open status and have labels CF and Open for Pickup.

[ben-mathu]

I cannot login using localhost:8080
I have tried to use the local.topcoder-dev.com, but I cannot see the top bar, navbar is hidden.Could someone help with this?

[flaming-cl]

@bug-bash-helper pull request is ready for review

[bug-bash-hunt-helper]

@flaming-cl ✅ this issue is marked as Ready for Review.

Now you may pick up another issue which is open for pickup if you like to.

[LieutenantRoger]

1. It's not a good idea to retrieve the challenge detail info one more time only to verify the member is regsitered in a challenge.

https://api.topcoder-dev.com/v5/resources?challengeId={challengeId}

I think you should relay on this API endpoint, which can get the resources of a challenge directly to see if the member has unregistered the challenge, so we avoid to reload challenge detail and other API data which we don't need.

[flaming-cl]

Hi, thanks so much for your feedback.
The pull request is updated with the suggestions you've made.
@bug-bash-helper pr is ready for review

[bug-bash-hunt-helper]

@flaming-cl ✅ this issue is marked as Ready for Review.

Now you may pick up another issue which is open for pickup if you like to.

[LieutenantRoger]

Hmmm I don't think the new PR is correct.

You called the getChallenge this time right ? I believe it's a different endpoint with the resource API. So we should only need to call resource API to verify whether a member has registered a challenge or not.

[flaming-cl]

@LieutenantRoger Sorry that I mistakenly thought "resources?challengeId={challengeId}" is for fetching the challenge resource. The API will just retrieve the registrants data. Thanks for explaining.

[flaming-cl]

@bug-bash-helper pr is ready for review

[bug-bash-hunt-helper]

@flaming-cl ✅ this issue is marked as Ready for Review.

Now you may pick up another issue which is open for pickup if you like to.

[codejamtc]

Fixed 🆗

bandicam.2021-11-28.13-06-39-142.mp4

[codejamtc]

Verified in Prod 🆗

bandicam.2021-11-29.17-40-40-451.mp4