Remark command#5

Merged

limyongsong merged 11 commits intomasterfrom Remark_command

Oct 11, 2017

+474-14

Commits on Oct 10, 2017

Remark Command: Step1 without the tests (from developerguide)
limyongsong
committed

Commits on Oct 11, 2017

Remark Command: Step 2 to make Remark command accept more inputs (i.e remark 1 r/ instead of remark) w/ RemarkCommandParser that helps parse/token string inputs and also update some tests
limyongsong
committed
RemarkCommand: Step 3 updating UI portion to have a label bar for remarks
limyongsong
committed
Remark Command: Step 4, Add a Remark Class which we will use instead of String, and replace all String usages with Remark class instead, and add simple RemarkTest
limyongsong
committed
Remark Command: Step 5, Use Remark Class in ReadOnlyPerson.java (i.e allow support) through the implementation Person.java, user will not be able to use the add and edit commands to modify the rema…
limyongsong
committed
Remark Command: Step 6, Add Remark field to XmlAdaptedPerson class, and fix tests to have remarks
limyongsong
committed
Remark Command: Edit personbuilder to be able to add remarks and update some tests
limyongsong
committed
Remark Command: Step 7, Allow UI label to read remarks data and also update the tests accordingly
limyongsong
committed
Remark Command: Step 8, Implement to logic that allows remarks command to work and connect all the components together and also add new test to test the changes
limyongsong
committed
Remark Command: XmlAdaptedPerson.java line 77 had bug (illegalvalueexception/ null exception), but seems ok after retrying, so added this.remark back so that values can be shown on UI
limyongsong
committed
Remark Command: RemarkCommandTest.java, line 64 showFirstPersonOnly(model); seems to be causing the model and expectedmodel comparison to have errors, so it is removed
limyongsong
committed