search

CS2103-AY1819S2-T12-3 / main

MediTabs: Medicine Stock Taking Application
https://cs2103-ay1819s2-t12-3.github.io/main/
MIT License
2 stars 4 forks source link

Adding Update command #42

Closed jtankw3 closed 5 years ago

jtankw3 commented 5 years ago
  1. Added basic Update command.
  2. Changed Add and Edit command functionality to make way for Update function: Add and Edit can no longer affect Quantity and Expiry fields. These fields must be changed with Update function to ensure synchronization between batch information and overall medicine information.
  3. Update all tests to take the above changes into account.
flxffy commented 5 years ago

Hi, can you add the Quantity and Expiry fields back into the code? (commit hash: a0d5596). It's not going to be possible to filter those fields for the warning panel if they don't exist. Thanks.

JonathanLeeWH commented 5 years ago

@flxffy Each batch of a particular have its own unique expiry date and quantity as each batch are usually manufactured at different times by the supplier. To get the quantity and expiry dates you can refer to the Batch.java class Commit: 5bf3653

On the side note, can @flxffy can you help me review and approve my PR? Thank you.