This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

#824 Update inventory quantity to not allow negative numbers, acceptance test added #848

Merged

jkleinsc merged 1 commit into HospitalRun:master from marcorosas1991:master

Dec 9, 2016

Contributor

marcorosas1991 commented Dec 9, 2016

Fixes #824.

Changes proposed in this pull request:

Updated inventory model to not allow negative numbers.
Updated inv-purchase model to not allow negative numbers.
Updated quantity-conv.js to consider negative quantities as an invalid number.
Added an acceptance test to make sure items with negative quantities cannot be added.
UI notifies when quantity is negative

cc @HospitalRun/core-maintainers


          HospitalRun#824 Update inventory quantity to not allow negative numbe…

d8f0226

…rs, added test

jkleinsc approved these changes

View reviewed changes

Member

jkleinsc left a comment

Looks good to me. Thanks for the PR @marcorosas1991!

jkleinsc merged commit 78ec35d into HospitalRun:master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet