Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

[$10] We should not be able to set "Working Days" value less then "daysPaid" #26

Closed
maxceem opened this issue Jun 16, 2021 · 4 comments
Closed

[$10] We should not be able to set "Working Days" value less then "daysPaid" #26

maxceem opened this issue Jun 16, 2021 · 4 comments
Labels
ACCEPTED PAID QA Pass on Prod
Milestone
v1.0.1 - Work Per...

Comments

@maxceem
Copy link
Contributor

maxceem commented Jun 16, 2021

When we update the value inside the "Working days" field, at the moment the minimal value is 0. Though we added one more field to the Work Periods daysPaid. We should not be able to set value in the "Working Days" less then "daysPaid".

Don't forget to list this field in the fields to get it form the server.
@maxceem maxceem added this to the v1.0 - Initial Launch milestone Jun 16, 2021
@maxceem maxceem changed the title We should not be able to set "Working Days" value less then "daysPaid" [$10] We should not be able to set "Working Days" value less then "daysPaid" Jun 16, 2021
@MadOPcode MadOPcode mentioned this issue Jun 17, 2021
Merged
@maxceem
Copy link
Contributor Author

maxceem commented Jun 17, 2021

Implemented and works good.

@maxceem
Copy link
Contributor Author

maxceem commented Jun 19, 2021

paid via https://www.topcoder.com/challenges/9314936a-a802-4eb8-b654-bf61be619582

@maxceem maxceem added the PAID label Jun 19, 2021
@sandhiyakavi
Copy link
Collaborator

Verified on Dev Env. Not able to set value less than daysPaid.

image

@sandhiyakavi
Copy link
Collaborator

Verified on PRod Env with @nkumar-topcoder

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
ACCEPTED PAID QA Pass on Prod
Projects
None yet
Milestone
v1.0.1 - Work Period Payments Improve...
Development

No branches or pull requests
2 participants
@maxceem @sandhiyakavi