-
Notifications
You must be signed in to change notification settings - Fork 24
[$50] ResourceBookings Edit UI: validation rules #85
Comments
Challenge https://www.topcoder.com/challenges/1a3c6e3d-d552-40aa-ae15-38db93311a28 has been created for this ticket. |
@bug-bash-helper assign |
@dashu-baba ✅ 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:
|
Challenge https://www.topcoder.com/challenges/1a3c6e3d-d552-40aa-ae15-38db93311a28 has been assigned to nowshad.bappy. |
Ready for Review": "@bug-bash-helper {https://github.com/topcoder-platform/taas-app/pull/96} is ready for review |
@dashu-baba ✅ this issue is marked as Now you may pick up another issue which is open for pickup if you like to. |
Okay I will update accordingly. |
@bug-bash-helper #96 is ready for review |
@dashu-baba ✅ this issue is marked as Now you may pick up another issue which is open for pickup if you like to. |
The payment for this task is increased to $50. But we would like to have a solution that properly utilizes Final Form features without workarounds. |
@maxceem If we made any changes in the Resource Booking, the changes are getting saved but we are not able to open the Edit Resource booking page again. It's loading continuously. issue.85.mp4 |
@sandhiyakavi this is fixed now. It was another issue which I've just merged #90
If you edit only one field it would show only one error for that field. But if you edit 2 fields, then it would show this error for 2 fields, because actually, we can fix this error by editing any of these fields, not only one of them. If user edits 2 fields we don't know which of the fields user has to fix that's why error is showing for both.
I've fixed so it is showing good on desktop screen: But on smaller screens, there might be not enough space, so it would be shown 2 lines anyway: |
Payment failed: [400]: Failed to activate challenge. Detail: Cannot Activate this project, it has no active billing accounts. |
Payment task has been updated: https://www.topcoder.com/challenges/1a3c6e3d-d552-40aa-ae15-38db93311a28 |
@lakshmiathreya Need to check this issue with Booking Manager/Admin role. Below features needs to be tested in edit Resource Booking page,
|
Uh oh!
There was an error while loading. Please reload this page.
Follow up from: #56 (comment)
If the Member Rate is a decimal value it is showing validation error. But we are able to give decimal values through API. So we have to fix UI validation and allow decimal numbers
Show validation error if we enter start date after end date, or end date after start date:
Start Date should not be after End Date
The error should have the same style as "title" error in Job Edit page
We can show it under each field if it's easier:
Start Date should not be after End Date
End Date should not be before Date
"Start Date" === "End Date" is allowed
The text was updated successfully, but these errors were encountered: