-
-
Notifications
You must be signed in to change notification settings - Fork 403
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
OBPIH-2416 Add validation for expiration dates #1353
Conversation
8d51649
to
e9cbe00
Compare
grails-app/controllers/org/pih/warehouse/inventory/InventoryItemController.groovy
Outdated
Show resolved
Hide resolved
42232c1
to
3398f16
Compare
@jmiranda I have updated PR and applied requested changes. |
grails-app/controllers/org/pih/warehouse/inventory/InventoryItemController.groovy
Outdated
Show resolved
Hide resolved
@@ -2469,6 +2469,7 @@ react.stockMovement.error.enterQuantity.label=Enter proper quantity | |||
react.stockMovement.error.boxWithoutPallet.label=Please enter Pack level 1 before Pack level 2. | |||
react.stockMovement.error.revertRequisitionItem.label=Could not revert requisition items | |||
react.stockMovement.error.futureDate.label=The date cannot be in the future | |||
react.stockMovement.error.invalidDate.label=This date is invalid. Please enter a date after 2000. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as in comment form file InventoryItemController.groovy
@@ -2535,6 +2536,7 @@ react.partialReceiving.error.quantityShippedNegative.label=Quantity shipped can' | |||
react.partialReceiving.error.futureDate.label=The date cannot be in the future | |||
react.partialReceiving.error.dateBeforeShipment.label=The date cannot be before shipment date | |||
react.partialReceiving.error.quantityToReceiveNegative.label=Quantity to receive can't be negative | |||
react.partialReceiving.error.invalidDate.label=This date is invalid. Please enter a date after 2000. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as in comment form file InventoryItemController.groovy
@@ -1453,6 +1453,7 @@ react.stockMovement.error.enterQuantity.label=Entrer la quantité correcte | |||
react.stockMovement.error.boxWithoutPallet.label=Veuillez entrer le niveau d’emballage 1 avant le niveau d’emballage 2. | |||
react.stockMovement.error.revertRequisitionItem.label=Impossible de reprendre les article de réquisition | |||
react.stockMovement.error.futureDate.label=La date ne peut pas être dans le futur | |||
react.stockMovement.error.invalidDate.label=Cette date n'est pas valide. Veuillez indiquer une date après 2000. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as in comment form file InventoryItemController.groovy
@@ -1518,6 +1519,7 @@ react.partialReceiving.error.quantityShippedNegative.label=Quantité expediée n | |||
react.partialReceiving.error.futureDate.label=La date ne peut pas être dans le futur | |||
react.partialReceiving.error.dateBeforeShipment.label=La date ne peut pas être avant la date d’expédition | |||
react.partialReceiving.error.quantityToReceiveNegative.label=Quantité à recevoir ne peut pas être negative | |||
react.partialReceiving.error.invalidDate.label=Cette date n'est pas valide. Veuillez indiquer une date après 2000. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue as in comment form file InventoryItemController.groovy
grails-app/services/org/pih/warehouse/inventory/InventoryService.groovy
Outdated
Show resolved
Hide resolved
grails-app/controllers/org/pih/warehouse/inventory/InventoryItemController.groovy
Outdated
Show resolved
Hide resolved
3398f16
to
7bf1276
Compare
@DominikaHatala I'm going to merge this since you made my changes, but could you double check whether you made the changes requested by @awalkowiak as well. |
No description provided.