Skip to content
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

[CS2103-W14-2] Inventory Manager #31

Open
wants to merge 480 commits into
base: master
Choose a base branch
from

Conversation

NicholasCF
Copy link

Team members:

  • Fang Shaohua
  • Liu Chao
  • Pang Jia Da
  • Nicholas Cristian Fernando

LiuChao93 and others added 30 commits March 31, 2020 22:15
…te-s

added function to delete each good price pair in the supplier's list
* 'master' of https://github.com/AY1920S2-CS2103-W14-2/main:
  correct formatting
  Implement redo (#98)
  Add index for github-pages redirection and stylesheet
  Update test case
  Correct formatting for checkstyle
  correct formatting for checkstyle
  added function to delete each good price pair in the supplier's list
…-s-function

Update the edit-s function to enable editing good price pair
…ement-deletegoodcommand

Implement DeleteGoodCommand
* 'master' of https://github.com/AY1920S2-CS2103-W14-2/main: (21 commits)
  Fix checkstyle errors
  Implement requested changes.
  Fix checkstyle errors
  Add DeleteGoodCommandTest tests.
  Implement DeleteSupplierCommandParserTest tests
  Hook DeleteGoodCommand into AddressBookParser
  Create DeleteGoodCommandParser
  Create DeleteGoodCommand
  Rename DeleteCommandParser to DeleteSupplierCommandParser
  correct formatting
  correct formatting
  edit formatting
  correct formatting
  update formatting
  Commented away the test cases as the test cases will be updated later
  correct formatting
  correct static variable order
  correct formating
  correct code formatting for checkstyle
  Correct formatting
  ...
…nce-sellcommand

Enhance SellCommand to take in selling price per unit
Branch update dg for edit supplier and delete supplier
* Fix bug: BuyCommand cannot cause inventory overflow

* Make BuyCommand require supplier's name.

* Refactor Good class to hold a Name for it's supplier

* Make BuyCommand require supplier to sell the good being bought.

* Fix broken BuyCommandParser and BuyCommand tests after BuyCommand enhancement

* Fix checkstyle errors

* Implement requested changes

* Resolve merge conflicts
* Update DG

* Update Design

* Add Data encryption under Implementation

Co-authored-by: Nicholas Cristian Fernando <[email protected]>
* 'master' of https://github.com/AY1920S2-CS2103-W14-2/main:
  Shaohua/docs/dg (#109)
  Add use case for undo and redo (#111)
  Jiada/feature/enhance buycommand (#102)
  Update formatting
  Editted DG for edit-s and delete-s functions
  Fix breaking sell command tests and checkstyle errors.
  Enhance sell command to require selling price per unit sold
FangShaoHua94 and others added 30 commits April 13, 2020 17:25
* Add undo enhancement to project portfolio page

* Add main details to project protfolio page

* Add new PR to list of contributions

* Add PR

* Add PR

* Add DG extract to portfolio

* Add snippets to portfolio

* Add section description

* Add newline EOF
…4-2/main

# Conflicts:
#	docs/DeveloperGuide.adoc
#	docs/UserGuide.adoc
* Add effort section

* Add coverage achievement

* Add EOF newline
…4-2/main (#195)

# Conflicts:
#	docs/DeveloperGuide.adoc
#	docs/UserGuide.adoc
* Merge branch 'documentation' of https://github.com/AY1920S2-CS2103-W14-2/main

# Conflicts:
#	docs/DeveloperGuide.adoc
#	docs/UserGuide.adoc

* Update error in UG

* Remove non-existent list-g command (#197)

* Fix user guide errors (#199)

Co-authored-by: FangShaoHua94 <[email protected]>
Co-authored-by: FangShaohua94 <[email protected]>
…ciidoc-violation

Fix asciidoc violations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants