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

[W13-4] TutorHelper #71

Open
wants to merge 1,113 commits into
base: master
Choose a base branch
from

Conversation

lywjoel
Copy link

@lywjoel lywjoel commented Sep 21, 2018

@andlimey @SusterDrifter @ooihuiying @ongyz

Copy link

@parvathy-sudhir-pillai parvathy-sudhir-pillai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good effort, team!

Suggestions to improve:

  • Most of the adocs still are titled as AddressBook - Level 4.
  • Remove unnecessary/dead links
  • Remove the header links which are actually from AB4.
  • There seems to be an issue with the rendering of use cases in the Developer Guide. Please have them formatted correctly. Use the netlify link to preview.
  • In the use cases, try to be generic and not specify that you have to TYPE a particular command, because then it makes your use cases apply only to your app and not any general Tutor Schedule maintainer app.
  • Update the user guide

Copy link

@parvathy-sudhir-pillai parvathy-sudhir-pillai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good effort, Team!
However, some of the areas need further improvement. Please take a look at the comments below to produce a better v1.2.

  • Readme adoc
  1. Should tweak or remove the top SE EDU nav bar, LOs link
  2. Product description still has AB-4's content from points 2-4. Update these.
  3. Travis CI build is failing. Check the log and rectify errors.
  4. Add AB-4 in Acknowledgements
  • Contact Us adoc
  1. Contact details, Issue tracker and PR links refer to AB-4. Please update.
  • User Guide adoc
  1. QuickStart refers to AB-4's jar file and instructions. Update this.
  2. Edit feature still refers to AB-4
  • Developer Guide doc
  1. Missing user stories. Add them
  2. More use cases can be added
  3. NFRs have some of the functional requirements. Remove these from NFR and add use cases, stories for these.


By: `Team SE-EDU` Since: `Jun 2016` Licence: `MIT`
By: `W13-4` Since: `September 2018` Licence: `Damith`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

License is still MIT

AddressBook Level 4 (AB4) is for those who *prefer to use a desktop app for managing contacts*. More importantly, AB4 is *optimized for those who prefer to work with a Command Line Interface* (CLI) while still having the benefits of a Graphical User Interface (GUI). If you can type fast, AB4 can get your contact management tasks done faster than traditional GUI apps. Interested? Jump to the <<Quick Start>> to get started. Enjoy!
TutorHelper is for tutors who have trouble managing their students’ details & progress, time of tuition and their own weekly tuition schedule. This project is an all-in-one scheduler.

System finds and deletes the student’s data.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this line mean? Seems out of place. Link it with the previous para.

=== Mark a part of syllabus(by index) as complete/incomplete: `mark`

Toggle an item completion stage of the selected student’s syllabus. +
Format: `mark STUDENT_INDEX SUBJECT_INDEX SYLLABUS_INDEX`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is unclear what values the subject and syllabus indices can take. If numbers what are they? What does a syllabus index of 3 mean?

1. MSS: Add Student

Step 1:
Actor: Tutor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actor should be specified before the MSS, not at Step 1.


Step 1:
Actor: Tutor
Action: Type command 'add' or 'a' followed by Student details on the CLI bar

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specifying that you have to "type" a command makes restricts your product to be that of a CLI-based input. Use cases need to be generic enough to be applicable to other types of input also. Typing is not a functional requirement.

Actor: Tutor
Action: Type command 'add' or 'a' followed by Student details on the CLI bar
Step 2a:
Input is valid and student details added into the database

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the same "active" voice when writing the way the system responds.
For e.g. System adds student details to the database.


== Non-Functional Requirements

. Display a list of upcoming sessions with students

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a non-functional requirement. It is very much functional, and similar to your list command


. Display a list of upcoming sessions with students
. Must be fast in displaying results
. Can view own timetable

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here. Not an NFR.

Copy link

@parvathy-sudhir-pillai parvathy-sudhir-pillai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feedback for v1.2

Good effort, Team! However, there are a few mistakes in the sequence diagrams provided in the dev-guide. Please go through the suggestions in the comments and pictures to improve for the next milestone.

Developer Guide

  • Generally, in all SDs, the green line coming to the LogicManager should start from outside the Logic component box.
  • We discourage low level diagrams that span multiple components (e.g. Between Logic and Model). Try to draw diagrams at multiple levels: one that shows component level interactions and others shown low level interactions within one components.
  • Group Students SD
    pic1
  • Students Payment SD
    image
  • Edit Payment SD
    image

Overall

  • Good job on adding/updating numerous UML diagrams. Please fix the issues pointed out.
  • Let's push the product towards a user-testable state by v1.3

Use case ends.
Extensions:
2a. Index is out of bounds
2a1. System displays error message.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please specify if the use case ends here or resumes at a specific step from the MSS. The same comments applies to all further use cases. For extensions specify what happens after the steps are carried out.

2. System adds student details into the database.
Use case ends.
Extensions:
2a. Index is out of bounds

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Index out of bounds for add student? From the specs of your add command, it doesn't seem to need an index. So what does this exception mean?


Step 2. The user executes `delete 5` command to delete the 5th person in the address book. The `delete` command calls `Model#commitAddressBook()`, causing the modified state of the address book after the `delete 5` command executes to be saved in the `addressBookStateList`, and the `currentStatePointer` is shifted to the newly inserted address book state.
* `AddressBookParser` -- Creates a GroupCommandParser object and parses the user's input.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still AddressBookParser? Please consider renaming and refactoring methods in your project as yours is no longer an AddressBook.


The following sequence diagram shows how the group operation works:

image::GroupSequenceDiagram.png[width="650"]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the sequence diagram for the group feature, it looks like the Model object was created later on after the command was issued by the Parser to the GroupCommand. Please refer to the AB-4 sample sequence diagram to see that the Model object should be existing already along with the LogicManager and AddressBookParser objects.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use larger/clearer pictures? The text in the picture is hard to read.


=== Students payment feature

1. Current implementation

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The formatting style for each of the features seem to be different. Please try to follow a uniform style throughout the document.

Bellaaarh pushed a commit to Bellaaarh/addressbook-level4 that referenced this pull request Oct 30, 2018
pangjiahao pushed a commit to pangjiahao/cs2103-u-schedule that referenced this pull request Oct 31, 2018
* Initial commit for updated GUI

* Fix to styling of subject content

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add TuitionTimingTest

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Add TuitionTimingContainsKeywordsPredicateTest

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Add EarningsParserTest and EarningsTest

* Update changes (#20)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Change logic/commands/CopySubCommand.java: copying to the same person is not allowed

* Change logic/commands/CopySubCommand.java: checks for subject index out of bound

* Add Messages.java: additional messages; Change logic/commands/CopySubCommand.java, EraseSyllCommand.java, MarkCommand.java: checks for invalid indexes

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add more error messages for invalid subject index and syllabus index

* Add CopySubCommandTest.java

* Fix to logic/commands/CopySubCommandTest.java

* Fix to style violation

* Update: pull from main repo (#24)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Edit DeveloperGuide.adoc: include edit syll documentation

* Edit DeveloperGuide.adoc: add edit syllabus feature to use cases

* Update changes (#17)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Update changes (#18)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Add EditSyllCommandTest.java: tests for duplicate syllabus and invalid indexes

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add EditSyllCommandParserTest.java

* Edit typicalPersonsAddressBook.xml: include syllabus

* Fixes to EditSyllCommandTest: changes to how out of bound syllabus index is obtained and messages

* Add TuitionTimingTest

* Add TuitionTimingContainsKeywordsPredicateTest

* Fixes to style violation

* Fixes to style violation

* Update: pull from main repo (#25)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix to display test

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Edit TypicalPerson.java: fix XmlSerializableAddressBookTest

* fix checkstyle

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Update changes (#21)

* Add CopySubCommandSystemTest.java

* fix checkstyle

* Change logic/commands/AppendSyllCommand.java: error checks for invalid subject added, Add logic/comands/AppendSyllCommandTest.java

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Add EditSyllCommandTest.java

* Add GroupCommandTest and fix errors

* Edit AppendSyllCommandTest.java: fix addressbook to tutorhelper

* Edit CopySubCommandSystemTest.java: fix addressbook to tutorhelper

* Edit GroupCommand.java: fix merge conlficts

* Add EditSyllCommandSystemTest.java and fix EditCommandSystemTest.java

* Edit EditSyllCommandSystemTest.java: remove unused imports

* Edit GroupCommand.java: fix unused import

* Edit imports to fix checkstyle

* Fix checkstyle errors

* Fix checkstyle errors: missing line at EOF

* Fix checkstyle errors: remove unused imports

* Update README.doc: change link for appveyor from lywjoel to CS2103

* Reverted changes to README.doc: link to appveyor is not found

* Update README.adoc

* Improve code readability and enhance minor tests

* Fix checkstyle

* Fix checkstyle

* Add AppendSyllCommandSystemTest.java

* Fix to testutil/TypicalPersons.java and systemtests/TutorHelperSystemTest.java: remove redundant code

* Fix systemtests/EditCommandSystemTest.java: remove redundant code

* Fix checkstyle

* Fix to style violation

* Fix AppendSyllCommandSystemTest.java: fix to style violation

* Add GroupCommandSystemTest

* Fix checkstyle violations

* Add SubjectsUtilTest

* Make Command Parsers inherit from Parser class

* Change CommandTestUtil.java: VALID_SYLLABUS to VALID_SYLLABUS_DIFFERENTIATION

* Fix typos

* Add AddSubCommandParserTest.java

* Add DeleteSubCommandParserTest.java

* Fix to style violation

* Fix typos

* Check for non-null parameters for AddSubCommand

* Change Subject equality conditions

* Modify SubjectTest to reflect new equality conditions

* Add AddSubCommandTest.java

* Add subject to Carl

* Redirect Exception message for DeleteSubCommand

* Add DeleteSubCommandTest.java

* Fix javadoc in AppendSyllCommandSystemTest.java

* Fix EraseSyllCommandSystemTest.java

* Add AddSubCommandSystemTest.java

* Add subject to Daniel

* Modify PayCommandSystemTest for Daniel

* Add subject to Daniel

* Add DeleteSubCommandSystemTest.java

* Update version number

* Remove gitter badge

* Refactor files from AddressBook to TutorHelper

* Update UserGuide.adoc

* Update Userguide.adoc

* Update DeveloperGuide.adoc

* Update DeveloperGuide.adoc

* Update version number

* Replace WebView of GUI with AnchorPane

* Rework UserGuide.adoc

* Replace "Address Book" with "TutorHelper"

* Change command descriptions

* Fix minor typos

* Update help page

* Add Appveyor under CI support

* Update UI image

* Add newline character to end of file

* Remove "include" tags

* Update UI.png

* Rework GUI

* Fix TuitionTiming bug to ensure that only 12hr timings are valid

* Fix test cases related to TuitionTiming

* Add new test case to TuitionTiming

* Change phone regex to only accept 8 digits and fix corresponding test cases

* Change name regex to accept only alphabetical inputs and fix the corresponding test cases

* Edit PayCommandTest.java: Update changes to edit paid command

* Edit documentation for GroupCommand and add assertions to sortByTime() function

* Edit documentation for TuitionTiming

* Edit Person.java: fix isSamePerson

* Edit UserGuide.adoc: restrictions to add command

* Restore new GUI

* Edit EditCommand.java: fix edit replace payments field

* Edit UserGuide.adoc: include remarks for index less then displayed list

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* Update UserGuide.adoc to reflect renaming of AppendSyll and EraseSyll

* Update UserGuide.adoc and add Personal Portfolio

* Update UserGuide.adoc

* Update AboutUs.adoc

* Update andlimey.adoc

* Add ooihuiying.adoc

* Fix to use only lowecase

* Fix to payment display

* Fix to UI: hide background initially

* Update: pull from main repo (#31)

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Add Newline Characters to End of Files

* Fix Import Statements Order

* Fix Import Statements Order

* Fix Code Style Violations

* Add 'group' Tag to UserGuide.adoc

* Add Tags to DeveloperGuide.adoc

* Rename Instances of AppendSyll and EraseSyll

* Fix AddSyll / DeleteSyll Command Words

* Update: pull from main repo (#32)

* Initial commit for updated GUI

* Fix to styling of subject content

* Fix to payment display

* Fix to UI: hide background initially

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Update changes (#28)

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemT…
Copy link

@parvathy-sudhir-pillai parvathy-sudhir-pillai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mid v1.4 Review

  • Update your About Us pages with PPP links
  • User Guide command summary, group command example shows Group Students : group TIMING
    e.g group Monday OR group 12pm OR group

    It feels like the command does allow a union operation. You may want to change the example or split it as multiple cases of group command.
  • In the dev guide, split the sequence diagrams for each feature into multiple levels. At the first level, show higher interactions between components. In the second level, show interactions within each component, but do not include the interactions between components. For e.g., the group command sequence diagram shows lower-level interaction between Logic and Model components. Please avoid this and use multi-level interaction diagrams.
    monster

All the best for your final presentation and demo

anthony-halim and others added 15 commits November 7, 2018 23:51
* Initial commit for updated GUI

* Fix to styling of subject content

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add TuitionTimingTest

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Add TuitionTimingContainsKeywordsPredicateTest

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Add EarningsParserTest and EarningsTest

* Update changes (#20)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Change logic/commands/CopySubCommand.java: copying to the same person is not allowed

* Change logic/commands/CopySubCommand.java: checks for subject index out of bound

* Add Messages.java: additional messages; Change logic/commands/CopySubCommand.java, EraseSyllCommand.java, MarkCommand.java: checks for invalid indexes

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add more error messages for invalid subject index and syllabus index

* Add CopySubCommandTest.java

* Fix to logic/commands/CopySubCommandTest.java

* Fix to style violation

* Update: pull from main repo (#24)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Edit DeveloperGuide.adoc: include edit syll documentation

* Edit DeveloperGuide.adoc: add edit syllabus feature to use cases

* Update changes (#17)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Update changes (#18)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Add EditSyllCommandTest.java: tests for duplicate syllabus and invalid indexes

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add EditSyllCommandParserTest.java

* Edit typicalPersonsAddressBook.xml: include syllabus

* Fixes to EditSyllCommandTest: changes to how out of bound syllabus index is obtained and messages

* Add TuitionTimingTest

* Add TuitionTimingContainsKeywordsPredicateTest

* Fixes to style violation

* Fixes to style violation

* Update: pull from main repo (#25)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix to display test

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Edit TypicalPerson.java: fix XmlSerializableAddressBookTest

* fix checkstyle

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Update changes (#21)

* Add CopySubCommandSystemTest.java

* fix checkstyle

* Change logic/commands/AppendSyllCommand.java: error checks for invalid subject added, Add logic/comands/AppendSyllCommandTest.java

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Add EditSyllCommandTest.java

* Add GroupCommandTest and fix errors

* Edit AppendSyllCommandTest.java: fix addressbook to tutorhelper

* Edit CopySubCommandSystemTest.java: fix addressbook to tutorhelper

* Edit GroupCommand.java: fix merge conlficts

* Add EditSyllCommandSystemTest.java and fix EditCommandSystemTest.java

* Edit EditSyllCommandSystemTest.java: remove unused imports

* Edit GroupCommand.java: fix unused import

* Edit imports to fix checkstyle

* Fix checkstyle errors

* Fix checkstyle errors: missing line at EOF

* Fix checkstyle errors: remove unused imports

* Update README.doc: change link for appveyor from lywjoel to CS2103

* Reverted changes to README.doc: link to appveyor is not found

* Update README.adoc

* Improve code readability and enhance minor tests

* Fix checkstyle

* Fix checkstyle

* Add AppendSyllCommandSystemTest.java

* Fix to testutil/TypicalPersons.java and systemtests/TutorHelperSystemTest.java: remove redundant code

* Fix systemtests/EditCommandSystemTest.java: remove redundant code

* Fix checkstyle

* Fix to style violation

* Fix AppendSyllCommandSystemTest.java: fix to style violation

* Add GroupCommandSystemTest

* Fix checkstyle violations

* Add SubjectsUtilTest

* Make Command Parsers inherit from Parser class

* Change CommandTestUtil.java: VALID_SYLLABUS to VALID_SYLLABUS_DIFFERENTIATION

* Fix typos

* Add AddSubCommandParserTest.java

* Add DeleteSubCommandParserTest.java

* Fix to style violation

* Fix typos

* Check for non-null parameters for AddSubCommand

* Change Subject equality conditions

* Modify SubjectTest to reflect new equality conditions

* Add AddSubCommandTest.java

* Add subject to Carl

* Redirect Exception message for DeleteSubCommand

* Add DeleteSubCommandTest.java

* Fix javadoc in AppendSyllCommandSystemTest.java

* Fix EraseSyllCommandSystemTest.java

* Add AddSubCommandSystemTest.java

* Add subject to Daniel

* Modify PayCommandSystemTest for Daniel

* Add subject to Daniel

* Add DeleteSubCommandSystemTest.java

* Update version number

* Remove gitter badge

* Refactor files from AddressBook to TutorHelper

* Update UserGuide.adoc

* Update Userguide.adoc

* Update DeveloperGuide.adoc

* Update DeveloperGuide.adoc

* Update version number

* Replace WebView of GUI with AnchorPane

* Rework UserGuide.adoc

* Replace "Address Book" with "TutorHelper"

* Change command descriptions

* Fix minor typos

* Update help page

* Add Appveyor under CI support

* Update UI image

* Add newline character to end of file

* Remove "include" tags

* Update UI.png

* Rework GUI

* Fix TuitionTiming bug to ensure that only 12hr timings are valid

* Fix test cases related to TuitionTiming

* Add new test case to TuitionTiming

* Change phone regex to only accept 8 digits and fix corresponding test cases

* Change name regex to accept only alphabetical inputs and fix the corresponding test cases

* Edit PayCommandTest.java: Update changes to edit paid command

* Edit documentation for GroupCommand and add assertions to sortByTime() function

* Edit documentation for TuitionTiming

* Edit Person.java: fix isSamePerson

* Edit UserGuide.adoc: restrictions to add command

* Restore new GUI

* Edit EditCommand.java: fix edit replace payments field

* Edit UserGuide.adoc: include remarks for index less then displayed list

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* Update UserGuide.adoc to reflect renaming of AppendSyll and EraseSyll

* Update UserGuide.adoc and add Personal Portfolio

* Update UserGuide.adoc

* Update AboutUs.adoc

* Update andlimey.adoc

* Add ooihuiying.adoc

* Fix to use only lowecase

* Fix to payment display

* Fix to UI: hide background initially

* Update: pull from main repo (#31)

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Add Newline Characters to End of Files

* Fix Import Statements Order

* Fix Import Statements Order

* Fix Code Style Violations

* Add 'group' Tag to UserGuide.adoc

* Add Tags to DeveloperGuide.adoc

* Rename Instances of AppendSyll and EraseSyll

* Fix AddSyll / DeleteSyll Command Words

* Update: pull from main repo (#32)

* Initial commit for updated GUI

* Fix to styling of subject content

* Fix to payment display

* Fix to UI: hide background initially

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Update changes (#28)

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemT…
Update AddressBook context to TutorHelper and fix test cases
* Initial commit for updated GUI

* Fix to styling of subject content

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add TuitionTimingTest

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Add TuitionTimingContainsKeywordsPredicateTest

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Add EarningsParserTest and EarningsTest

* Update changes (#20)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Change logic/commands/CopySubCommand.java: copying to the same person is not allowed

* Change logic/commands/CopySubCommand.java: checks for subject index out of bound

* Add Messages.java: additional messages; Change logic/commands/CopySubCommand.java, EraseSyllCommand.java, MarkCommand.java: checks for invalid indexes

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add more error messages for invalid subject index and syllabus index

* Add CopySubCommandTest.java

* Fix to logic/commands/CopySubCommandTest.java

* Fix to style violation

* Update: pull from main repo (#24)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Edit DeveloperGuide.adoc: include edit syll documentation

* Edit DeveloperGuide.adoc: add edit syllabus feature to use cases

* Update changes (#17)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Update changes (#18)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Add EditSyllCommandTest.java: tests for duplicate syllabus and invalid indexes

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add EditSyllCommandParserTest.java

* Edit typicalPersonsAddressBook.xml: include syllabus

* Fixes to EditSyllCommandTest: changes to how out of bound syllabus index is obtained and messages

* Add TuitionTimingTest

* Add TuitionTimingContainsKeywordsPredicateTest

* Fixes to style violation

* Fixes to style violation

* Update: pull from main repo (#25)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix to display test

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Edit TypicalPerson.java: fix XmlSerializableAddressBookTest

* fix checkstyle

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Update changes (#21)

* Add CopySubCommandSystemTest.java

* fix checkstyle

* Change logic/commands/AppendSyllCommand.java: error checks for invalid subject added, Add logic/comands/AppendSyllCommandTest.java

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Add EditSyllCommandTest.java

* Add GroupCommandTest and fix errors

* Edit AppendSyllCommandTest.java: fix addressbook to tutorhelper

* Edit CopySubCommandSystemTest.java: fix addressbook to tutorhelper

* Edit GroupCommand.java: fix merge conlficts

* Add EditSyllCommandSystemTest.java and fix EditCommandSystemTest.java

* Edit EditSyllCommandSystemTest.java: remove unused imports

* Edit GroupCommand.java: fix unused import

* Edit imports to fix checkstyle

* Fix checkstyle errors

* Fix checkstyle errors: missing line at EOF

* Fix checkstyle errors: remove unused imports

* Update README.doc: change link for appveyor from lywjoel to CS2103

* Reverted changes to README.doc: link to appveyor is not found

* Update README.adoc

* Improve code readability and enhance minor tests

* Fix checkstyle

* Fix checkstyle

* Add AppendSyllCommandSystemTest.java

* Fix to testutil/TypicalPersons.java and systemtests/TutorHelperSystemTest.java: remove redundant code

* Fix systemtests/EditCommandSystemTest.java: remove redundant code

* Fix checkstyle

* Fix to style violation

* Fix AppendSyllCommandSystemTest.java: fix to style violation

* Add GroupCommandSystemTest

* Fix checkstyle violations

* Add SubjectsUtilTest

* Make Command Parsers inherit from Parser class

* Change CommandTestUtil.java: VALID_SYLLABUS to VALID_SYLLABUS_DIFFERENTIATION

* Fix typos

* Add AddSubCommandParserTest.java

* Add DeleteSubCommandParserTest.java

* Fix to style violation

* Fix typos

* Check for non-null parameters for AddSubCommand

* Change Subject equality conditions

* Modify SubjectTest to reflect new equality conditions

* Add AddSubCommandTest.java

* Add subject to Carl

* Redirect Exception message for DeleteSubCommand

* Add DeleteSubCommandTest.java

* Fix javadoc in AppendSyllCommandSystemTest.java

* Fix EraseSyllCommandSystemTest.java

* Add AddSubCommandSystemTest.java

* Add subject to Daniel

* Modify PayCommandSystemTest for Daniel

* Add subject to Daniel

* Add DeleteSubCommandSystemTest.java

* Update version number

* Remove gitter badge

* Refactor files from AddressBook to TutorHelper

* Update UserGuide.adoc

* Update Userguide.adoc

* Update DeveloperGuide.adoc

* Update DeveloperGuide.adoc

* Update version number

* Replace WebView of GUI with AnchorPane

* Rework UserGuide.adoc

* Replace "Address Book" with "TutorHelper"

* Change command descriptions

* Fix minor typos

* Update help page

* Add Appveyor under CI support

* Update UI image

* Add newline character to end of file

* Remove "include" tags

* Update UI.png

* Rework GUI

* Fix TuitionTiming bug to ensure that only 12hr timings are valid

* Fix test cases related to TuitionTiming

* Add new test case to TuitionTiming

* Change phone regex to only accept 8 digits and fix corresponding test cases

* Change name regex to accept only alphabetical inputs and fix the corresponding test cases

* Edit PayCommandTest.java: Update changes to edit paid command

* Edit documentation for GroupCommand and add assertions to sortByTime() function

* Edit documentation for TuitionTiming

* Edit Person.java: fix isSamePerson

* Edit UserGuide.adoc: restrictions to add command

* Restore new GUI

* Edit EditCommand.java: fix edit replace payments field

* Edit UserGuide.adoc: include remarks for index less then displayed list

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* Update UserGuide.adoc to reflect renaming of AppendSyll and EraseSyll

* Update UserGuide.adoc and add Personal Portfolio

* Update UserGuide.adoc

* Update AboutUs.adoc

* Update andlimey.adoc

* Add ooihuiying.adoc

* Fix to use only lowecase

* Fix to payment display

* Fix to UI: hide background initially

* Update: pull from main repo (#31)

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Add Newline Characters to End of Files

* Fix Import Statements Order

* Fix Import Statements Order

* Fix Code Style Violations

* Add 'group' Tag to UserGuide.adoc

* Add Tags to DeveloperGuide.adoc

* Rename Instances of AppendSyll and EraseSyll

* Fix AddSyll / DeleteSyll Command Words

* Update: pull from main repo (#32)

* Initial commit for updated GUI

* Fix to styling of subject content

* Fix to payment display

* Fix to UI: hide background initially

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Update changes (#28)

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemT…
Fix UI bugs for editsyll/addsyll/deletesyll/mark/paid
lywjoel and others added 28 commits November 12, 2018 21:26
* Refactor seedu.address Package to tutorhelper

* Refactor seedu.address Package to tutorhelper

* Update Documentation for Refactoring

* Fix Code Style Violations

* Fix Code Style Violations

* Fix Code Style Violations

* Refactor Import Statements

* Remove Unused Import from TuitionTiming.java

* Refactor Format Checkers into Parser Classes

* Update DeveloperGuide.adoc

* minor fix to payment

* modify paycommand

* Add Check before Loading Payments

* Update: pull from main repo (#37)

* Update andlimey.adoc

* Update Version Number

* Update ooihuiying.adoc

* Change toString() of Student

* Update UserGuide for AddSub

* Improve developer guide.

* Remove LearningOutcomes.adoc

* Remove DummySearchPage.html

* Fix Payment Command Example

* Changes to DeveloperGuide.adoc

* Update changes (#40)

* Update Ui.png in README.adoc

* Update: pull from main repo (#36)

* Edit images for DeveloperGuide.adoc

* Update changes (#37)

* Update: pull from main (#33)

* Initial commit for updated GUI

* Fix to styling of subject content

* Fix to payment display

* Fix to UI: hide background initially

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Tidy up Refactoring of AppendSyll / EraseSyll to AddSyll / DeleteSyll

* Fix Import Statements

* Update changes (#28)

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add TuitionTimingTest

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Add TuitionTimingContainsKeywordsPredicateTest

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Add EarningsParserTest and EarningsTest

* Update changes (#20)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Change logic/commands/CopySubCommand.java: copying to the same person is not allowed

* Change logic/commands/CopySubCommand.java: checks for subject index out of bound

* Add Messages.java: additional messages; Change logic/commands/CopySubCommand.java, EraseSyllCommand.java, MarkCommand.java: checks for invalid indexes

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add more error messages for invalid subject index and syllabus index

* Add CopySubCommandTest.java

* Fix to logic/commands/CopySubCommandTest.java

* Fix to style violation

* Update: pull from main repo (#24)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Edit DeveloperGuide.adoc: include edit syll documentation

* Edit DeveloperGuide.adoc: add edit syllabus feature to use cases

* Update changes (#17)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Update changes (#18)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Add EditSyllCommandTest.java: tests for duplicate syllabus and invalid indexes

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Add EditSyllCommandParserTest.java

* Edit typicalPersonsAddressBook.xml: include syllabus

* Fixes to EditSyllCommandTest: changes to how out of bound syllabus index is obtained and messages

* Add TuitionTimingTest

* Add TuitionTimingContainsKeywordsPredicateTest

* Fixes to style violation

* Fixes to style violation

* Update: pull from main repo (#25)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Add MarkCommandSystemTest.java and MarkCommandTest.java

* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented

* fix AddCommandSystemTest, AddCommandTest,

* Fix checkstyle

* Fix checkstyle

* Fix one line of checkstyle

* Fix checkstyle

* Add EraseSyllCommandSystemTest.java

* Fix to style violations

* Fixes to missing javadoc

* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters

* Fix XmlSerializableAddressBookTest

* Fix HelpCommandSystemTest.java

* Fix formatting of BrowserPanelTest.java to improve readability

* Increase GuiRobot.java timeout to account for improved GUI

* Fix BrowserPanelTest.java

* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix minor checkstyle

* Fix to display test

* Fix AddCommandSystemTest

* Add newline character to end of file

* Fix checkstyle

* Edit TypicalPerson.java: fix XmlSerializableAddressBookTest

* fix checkstyle

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Update changes (#21)

* Add CopySubCommandSystemTest.java

* fix checkstyle

* Change logic/commands/AppendSyllCommand.java: error checks for invalid subject added, Add logic/comands/AppendSyllCommandTest.java

* Refactor AddressBook to TutorHelper

* Fix some minor errors in refactoring

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add tests for GroupCommandParserTest

* Add EditSyllCommandTest.java

* Add GroupCommandTest and fix errors

* Edit AppendSyllCommandTest.java: fix addressbook to tutorhelper

* Edit CopySubCommandSystemTest.java: fix addressbook to tutorhelper

* Edit GroupCommand.java: fix merge conlficts

* Add EditSyllCommandSystemTest.java and fix EditCommandSystemTest.java

* Edit EditSyllCommandSystemTest.java: remove unused imports

* Edit GroupCommand.java: fix unused import

* Edit imports to fix checkstyle

* Fix checkstyle errors

* Fix checkstyle errors: missing line at EOF

* Fix checkstyle errors: remove unused imports

* Update README.doc: change link for appveyor from lywjoel to CS2103

* Reverted changes to README.doc: link to appveyor is not found

* Update README.adoc

* Improve code readability and enhance minor tests

* Fix checkstyle

* Fix checkstyle

* Add AppendSyllCommandSystemTest.java

* Fix to testutil/TypicalPersons.java and systemtests/TutorHelperSystemTest.java: remove redundant code

* Fix systemtests/EditCommandSystemTest.java: remove redundant code

* Fix checkstyle

* Fix to style violation

* Fix AppendSyllCommandSystemTest.java: fix to style violation

* Add GroupCommandSystemTest

* Fix checkstyle violations

* Add SubjectsUtilTest

* Make Command Parsers inherit from Parser class

* Change CommandTestUtil.java: VALID_SYLLABUS to VALID_SYLLABUS_DIFFERENTIATION

* Fix typos

* Add AddSubCommandParserTest.java

* Add DeleteSubCommandParserTest.java

* Fix to style violation

* Fix typos

* Check for non-null parameters for AddSubCommand

* Change Subject equality conditions

* Modify SubjectTest to reflect new equality conditions

* Add AddSubCommandTest.java

* Add subject to Carl

* Redirect Exception message for DeleteSubCommand

* Add DeleteSubCommandTest.java

* Fix javadoc in AppendSyllCommandSystemTest.java

* Fix EraseSyllCommandSystemTest.java

* Add AddSubCommandSystemTest.java

* Add subject to Daniel

* Modify PayCommandSystemTest for Daniel

* Add subject to Daniel

* Add DeleteSubCommandSystemTest.java

* Update version number

* Remove gitter badge

* Refactor files from AddressBook to TutorHelper

* Update UserGuide.adoc

* Update Userguide.adoc

* Update DeveloperGuide.adoc

* Update DeveloperGuide.adoc

* Update version number

* Replace WebView of GUI with AnchorPane

* Rework UserGuide.adoc

* Replace "Address Book" with "TutorHelper"

* Change command descriptions

* Fix minor typos

* Update help page

* Add Appveyor under CI support

* Update UI image

* Add newline character to end of file

* Remove "include" tags

* Update UI.png

* Rework GUI

* Fix TuitionTiming bug to ensure that only 12hr timings are valid

* Fix test cases related to TuitionTiming

* Add new test case to TuitionTiming

* Change phone regex to only accept 8 digits and fix corresponding test cases

* Change name regex to accept only alphabetical inputs and fix the corresponding test cases

* Edit PayCommandTest.java: Update changes to edit paid command

* Edit documentation for GroupCommand and add assertions to sortByTime() function

* Edit documentation for TuitionTiming

* Edit Person.java: fix isSamePerson

* Edit UserGuide.adoc: restrictions to add command

* Restore new GUI

* Edit EditCommand.java: fix edit replace payments field

* Edit UserGuide.adoc: include remarks for index less then displayed list

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* Update UserGuide.adoc to reflect renaming of AppendSyll and EraseSyll

* Update UserGuide.adoc and add Personal Portfolio

* Update UserGuide.adoc

* Update AboutUs.adoc

* Update andlimey.adoc

* Add ooihuiying.adoc

* Fix to use only lowecase

* Fixes to header not showing up due to tags

* Fixes to header not showing up due to tags

* Update andlimey.adoc and add assertion to GroupCommand

* Fix typos in team portfolios

Remove "Level 4"

* Reformat Links in AboutUs.adoc

* Add Personal Portfolio Page lywjoel.adoc

* Add Tags to Documentation for lywjoel.adoc

* Add Newline Characters to End of Files

* Fix Import Statements Order

* Fix Import Statements Order

* Fix Code Style Violations

* Add 'group' Tag to UserGuide.adoc

* Add Tags to DeveloperGuide.adoc

* Update changes (#29)

* Initial commit for updated GUI

* Fix to styling of subject content

* Restore new GUI

* Fix broken Help page image link

* Fix code style violations

* Fix netlify preview image issue

* minor edit to the number of payments displayed.

Changed it to 9 to display maximum of 10 instead of the original 11.

* Resolve issue for integer overflow for amount, month and year.

* Change earnings variable from int to long to account for cases where earnings is very large.

* Improve portion of the code in checking for integer overflow.

* Enhance tests for payment

* Enhance Earnings test

* fix checkstyle

* fix checkstyle

* fix checkstyle

* Refactor AppendSyll into AddSyll, EraseSyll into DeleteSyll

* Update: pull from main repo (#30)

* Update changes (#25)

* Add PayCommandSystemTest

* Add PayCommandSystemTest

* Fix checkstyle

* Update: pull from main repo (#28)

* Update changes (#19)

* Update: pull from main repo (#21)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Fix variable type mismatch

* Fix code style

* Fix code style

* Update: pull from main repo (#22)

* Edit PayCommand.java: refactored payment edit 

refactored payment edit with findPaymentToUpdate() and editPaymentField()

* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor

* Fix AddCommandParserTest

* Attempt at fixing errors with sequence diagram in developer docs.

* Edit Developer guide implementation

* Modify command style for both earnings and payment

* Fix Payment test

* Fix checkstyle

* Fix checkstyle

* Fix checkstyle

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Refine Group Command to sort based on day or time after filtering the list

* Fix DeveloperGuide.adoc merge conflicts

* Fix DeveloperGuide.adoc merge conflicts

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Fix typos in command files

* Add addsub command

* Fix more typos

* Add deletesub command

* Fix code style

* Update documentation for addsub / deletesub

* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java

* Change Subject and Syllabus constructor

* Change SubjectTypeTest.java: test method name

* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4

* Add newline character to end of file

* Fix code style

* Fixes to test cases

* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests

* Change model/subject/Syllabus.java: message constraint to better reflect restrictions

* Change address/testutil/TypicalIndexes.java: more indexes for usage

* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation

* Change logic/parser/CopySubCommandParser.java: description of the class

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Fix to performance test

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment method

* Edit Person.java: isSamePerson checks all identity fields

* Fix xml files to include tutitionTiming and subject

* Fix code style

* Fix code style

* Update: pull from main repo (#23)

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix AddCommandParserTest

* Fix merge conflicts for test cases

* Add EditSyllCommand/Parser.java: command to handle editing of syllabus

* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus

* Edit EditSyllCommand.java: fix error messages

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java

* Fix merge conflicts

* Edit XmlUtilTest.data/xml files: include payment entry

* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest

remove comparison for index in XmlAdaptedPay

* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest 

Remove payment in EditPersonDescriptor, edit function not used to edit payment fields

* Edit Person.java: isSamePerson considers all identity fields and test cases associated

* Fix merge conflicts for test cases

* Edit XmlAdaptedPay.java: fix equals

* Fix AddressBookParserTest

* Add newline characters to end of files

* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment

* Fix variable type mismatch

* Edit PersonUtil.java: include Subject import

* Edit EditPersonDecriptorBuilder: add emptyPayment me…
Update Personal Portfolio Page, Add HelpWindow.html
Update asciidoctor.css, Update Personal Portfolio Page
add link to ooihuiying.adoc
@lywjoel
Copy link
Author

lywjoel commented Nov 12, 2018

Netlify does not seem to be triggered for the last two PRs.
Our netlify link: https://deploy-preview-71--elated-hodgkin-792862.netlify.com/
The pages in the link are not updated to the latest versions.

To hide from google search results.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
7 participants