search

spooky-oysters / Bangazon-Command-Line-Application

0 stars 1 forks source link

Kb added date property to product #38

Closed kimberly-bird closed 6 years ago

kimberly-bird commented 6 years ago

Link to Ticket

4

Description of Proposed Changes

Steps to Test

Outline the steps to test

git fetch --all
git checkout kb-updateProductMenu
DELETE YOUR TEST DB
cd bangazon-cli.Tests
dotnet restore
dotnet test

Impacted Areas in Application

List general components of the application that this PR will affect:

Mentions @username

Tag users that need to review this code

Definition of Done

  1. The project must be fully documented. This includes the following:
    1. Complete README that documents the steps to install the code, how to install any dependencies, or system configuration needed.
    2. Every class must be documented with purpose, author, and methods.
    3. Every method must be documented with purpose and argument list - which itself must contain a short purpose for each argument.
  2. The project must be able to run fully, and without errors, on each teammate's system.
  3. Fulfills every requirement. Every line of code has been peer reviewed.
  4. For projects that require unit testing, core functionality must be identified and have at least one test for each.
krysmathis commented 6 years ago

👍

Chewieez commented 6 years ago

Your code comment on line 111 of ProductManager.cs needs to be changed to // clear existing products

Other than that, looks good!