-
Notifications
You must be signed in to change notification settings - Fork 33
[$70] Make some ResourceBooking fields not required. #84
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/30160479 has been created for this ticket. |
@imcaizheng this issue is open for pickup. |
Contest https://www.topcoder.com/challenges/30160479 has been updated - it has been assigned to aaron2017. |
PR created #86 |
I've updated the budget as we also need to create a migration script for this. Details in the review #86 (review) |
@nkumar-topcoder I would need your help to apply the fix for this issue. We need to run a migration script that we've already created. The migration script can be run by
So this would need to be run on both DEV and on PROD servers. Theoretically, we could add these commands to the CirecleCI, but I guess it would be better to keep it manual for better control for now. |
Tested on DEV, works good. |
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30160479 |
Make the following fields on
ResourceBooking
be not required during all the operations:Update documentation (swagger, postman) accordingly.
The text was updated successfully, but these errors were encountered: