-
Notifications
You must be signed in to change notification settings - Fork 27
Associate resource creation with challenge state. #15
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
@topcoder-platform/tcxcommunity this is open for pickup! Worked based on the |
@topcoder-platform/topcodercompetitors this is open for pickup! Worked based on the |
@ThomasKranitsas I would like to work on this, can you assign me as I cannot assign myself? |
Contest https://www.topcoder.com/challenges/30119420 has been created for this ticket. |
@tcchhabra, please sign-up with Topcoder X tool |
@ThomasKranitsas I have tried again using all 3 major browsers, safari/chrome and Firefox, I think this Topcoder x is broken. Can we use GitHub access management for ticket assignment ? It is a very reliable. I had created a ticket few weeks back and support also closed it without any help. |
The registration phase of the challenge is not open. It's not possible to access forum & use the link |
I have opened the registration phase manually |
@ThomasKranitsas Can I work on this issue? The self registraion link is broken hence don't have rights to assign this to myself |
Contest https://www.topcoder.com/challenges/30119420 has been updated - it has been assigned to codingdrone. |
Hey @codingdrone! Can we have an update please? |
Working, would be done in few hours. |
@ThomasKranitsas can you please provide some sample data for ResourceRolePhaseDependency table? Like sample data for Rosource table is provided in Resource.json |
Example:
The above would translate into: |
@ThomasKranitsas Its been more than a day, can i take it now? |
@codingdrone Status check. Are you on track to submit within the next 7 hours? If not, I am afraid I will have to reassign the ticket. Kindly give an update here. P.S. @ThomasKranitsas is still the CoPilot for the project. They are not feeling well and I am only helping them temporarily with the open items. |
Contest https://www.topcoder.com/challenges/30119420 has been updated - sadathanwar17 has been unassigned. |
Contest https://www.topcoder.com/challenges/30119420 has been updated - sadathanwar17 has been unassigned. |
Contest https://www.topcoder.com/challenges/30119420 has been updated - it has been assigned to sadathanwar17. |
@asadath1395 Status Check. Are you actively working on this and on track to submit? |
@callmekatootie Yes i am actively working. I might need another 12 hours since scope is quite big |
That's cool. Thanks for the update |
I am opening this up for others as i will not be able to complete soon |
This is no longer open for pickup. |
Currently, a resource can be created regardless of the challenge state. That would mean a member could register on a challenge even if the registration phase is not open.
As part of this ticket, you need to create a new table called
ResourceRolePhaseDependency
.The format of the objects stored on this will be as follows:
Detailed requirements
phaseId
should be validated by calling the Challenge API to fetch thePhases
(GET <Challenge API URL>/challengephases
) and theresourceRoleId
should be validated by fetching the availableResourceRoles
.seed-tables
script so we can insert initial data into the DB./resources
endpoint based on the logic described in the next section.Updated business logic for the
/resources
endpointWhen creating/updating/deleting a
Resource
we should:ResourceRole
of the newly createdResource
Let's take the member registration on a challenge as an example. The
ResourceRole
would depend on the RegistrationPhase
to be open (isOpen: true
).If the requirements are not met, throw a
400 Bad Request
with an appropriate error message (egThe resource cannot be created because the Registration phase is not open
).The text was updated successfully, but these errors were encountered: