{"push_id":3010249609,"size":53,"distinct_size":53,"ref":"refs/heads/master","head":"d047a6a91b48a91159bbed033724b5519f71a0f5","before":"84145534165777e39b83d8c689ece2802ef50f40","commits":[{"sha":"b2416bbf6282cdb0763771edcd0b2a6a42809a22","author":{"name":"Andy","email":"ee1ffd8d85b9ebed997a2cd3c9a3df88d7a74d37@users.noreply.github.com"},"message":"Merge pull request #29 from CS2103-AY1819S1-W13-4/master\n\nFix test cases and checkstyle violations","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/b2416bbf6282cdb0763771edcd0b2a6a42809a22"},{"sha":"b97de0e5ed98d11547e597c9feb3d8de24857b2e","author":{"name":"Anthony Halim","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Update: pull from main repo (#28)\n\n* Update changes (#19)\r\n\r\n* Update: pull from main repo (#21)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Fix variable type mismatch\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#22)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#23)\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Add MarkCommandSystemTest.java and MarkCommandTest.java\r\n\r\n* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented\r\n\r\n* fix AddCommandSystemTest, AddCommandTest,\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix one line of checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EraseSyllCommandSystemTest.java\r\n\r\n* Fix to style violations\r\n\r\n* Fixes to missing javadoc\r\n\r\n* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters\r\n\r\n* Fix XmlSerializableAddressBookTest\r\n\r\n* Fix HelpCommandSystemTest.java\r\n\r\n* Fix formatting of BrowserPanelTest.java to improve readability\r\n\r\n* Increase GuiRobot.java timeout to account for improved GUI\r\n\r\n* Fix BrowserPanelTest.java\r\n\r\n* Add TuitionTimingTest\r\n\r\n* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle\r\n\r\n* Add TuitionTimingContainsKeywordsPredicateTest\r\n\r\n* Fix minor checkstyle\r\n\r\n* Fix minor checkstyle\r\n\r\n* Fix minor checkstyle\r\n\r\n* Fix AddCommandSystemTest\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix checkstyle\r\n\r\n* Add EarningsParserTest and EarningsTest\r\n\r\n* Update changes (#20)\r\n\r\n* Update: pull from main repo (#21)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Fix variable type mismatch\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#22)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#23)\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Add MarkCommandSystemTest.java and MarkCommandTest.java\r\n\r\n* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented\r\n\r\n* fix AddCommandSystemTest, AddCommandTest,\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix one line of checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EraseSyllCommandSystemTest.java\r\n\r\n* Fix to style violations\r\n\r\n* Fixes to missing javadoc\r\n\r\n* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters\r\n\r\n* Change logic/commands/CopySubCommand.java: copying to the same person is not allowed\r\n\r\n* Change logic/commands/CopySubCommand.java: checks for subject index out of bound\r\n\r\n* Add Messages.java: additional messages; Change logic/commands/CopySubCommand.java, EraseSyllCommand.java, MarkCommand.java: checks for invalid indexes\r\n\r\n* Fix XmlSerializableAddressBookTest\r\n\r\n* Fix HelpCommandSystemTest.java\r\n\r\n* Fix formatting of BrowserPanelTest.java to improve readability\r\n\r\n* Increase GuiRobot.java timeout to account for improved GUI\r\n\r\n* Fix BrowserPanelTest.java\r\n\r\n* Add more error messages for invalid subject index and syllabus index\r\n\r\n* Add CopySubCommandTest.java\r\n\r\n* Fix to logic/commands/CopySubCommandTest.java\r\n\r\n* Fix to style violation\r\n\r\n* Update: pull from main repo (#24)\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Edit DeveloperGuide.adoc: include edit syll documentation\r\n\r\n* Edit DeveloperGuide.adoc: add edit syllabus feature to use cases\r\n\r\n* Update changes (#17)\r\n\r\n* Update: pull from main repo (#21)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Fix variable type mismatch\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* fix AddCommandSystemTest, AddCommandTest,\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix one line of checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Update changes (#18)\r\n\r\n* Update: pull from main repo (#21)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Fix variable type mismatch\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#22)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#23)\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Add MarkCommandSystemTest.java and MarkCommandTest.java\r\n\r\n* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented\r\n\r\n* fix AddCommandSystemTest, AddCommandTest,\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix one line of checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EraseSyllCommandSystemTest.java\r\n\r\n* Fix to style violations\r\n\r\n* Fixes to missing javadoc\r\n\r\n* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters\r\n\r\n* Fix XmlSerializableAddressBookTest\r\n\r\n* Add EditSyllCommandTest.java: tests for duplicate syllabus and invalid indexes\r\n\r\n* Fix HelpCommandSystemTest.java\r\n\r\n* Fix formatting of BrowserPanelTest.java to improve readability\r\n\r\n* Increase GuiRobot.java timeout to account for improved GUI\r\n\r\n* Fix BrowserPanelTest.java\r\n\r\n* Add EditSyllCommandParserTest.java\r\n\r\n* Edit typicalPersonsAddressBook.xml: include syllabus\r\n\r\n* Fixes to EditSyllCommandTest: changes to how out of bound syllabus index is obtained and messages\r\n\r\n* Add TuitionTimingTest\r\n\r\n* Add TuitionTimingContainsKeywordsPredicateTest\r\n\r\n* Fixes to style violation\r\n\r\n* Fixes to style violation\r\n\r\n* Update: pull from main repo (#25)\r\n\r\n* Update changes (#19)\r\n\r\n* Update: pull from main repo (#21)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Fix variable type mismatch\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#22)\r\n\r\n* Edit PayCommand.java: refactored payment edit \r\n\r\nrefactored payment edit with findPaymentToUpdate() and editPaymentField()\r\n\r\n* Edit DeveloperGuide.adoc: update sequence diagram after edit payment refactor\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Attempt at fixing errors with sequence diagram in developer docs.\r\n\r\n* Edit Developer guide implementation\r\n\r\n* Modify command style for both earnings and payment\r\n\r\n* Fix Payment test\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Refine Group Command to sort based on day or time after filtering the list\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Fix DeveloperGuide.adoc merge conflicts\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Fix typos in command files\r\n\r\n* Add addsub command\r\n\r\n* Fix more typos\r\n\r\n* Add deletesub command\r\n\r\n* Fix code style\r\n\r\n* Update documentation for addsub / deletesub\r\n\r\n* Add SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java\r\n\r\n* Change Subject and Syllabus constructor\r\n\r\n* Change SubjectTypeTest.java: test method name\r\n\r\n* Change SubjectTest.java, SubjectTypeTest.java, SyllabusTest.java: downgrade from JUnit 5 to JUnit 4\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix code style\r\n\r\n* Fixes to test cases\r\n\r\n* Add AppendSyllCommandParserTest.java, EraseSyllCommandParserTest.java, MarkCommandParserTest.java, CopySubCommandParserTest.java: tests\r\n\r\n* Change model/subject/Syllabus.java: message constraint to better reflect restrictions\r\n\r\n* Change address/testutil/TypicalIndexes.java: more indexes for usage\r\n\r\n* Fix logic/commands/EraseSyllCommand.java, MarkCommand.java: equal implementation\r\n\r\n* Change logic/parser/CopySubCommandParser.java: description of the class\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Fix to performance test\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Update: pull from main repo (#23)\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix AddCommandParserTest\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Add EditSyllCommand/Parser.java: command to handle editing of syllabus\r\n\r\n* Edit Subject/AddressBookParser.java: Edit functions to edit syllabus\r\n\r\n* Edit EditSyllCommand.java: fix error messages\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Update src/main/java/seedu/address/logic/parser/AddressBookParser.java\r\n\r\n* Fix merge conflicts\r\n\r\n* Edit XmlUtilTest.data/xml files: include payment entry\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix XmlUtilTest\r\n\r\nremove comparison for index in XmlAdaptedPay\r\n\r\n* Edit PersonUtil/EditPersonDescriptorBuilder.java: fix AddressBookParserTest \r\n\r\nRemove payment in EditPersonDescriptor, edit function not used to edit payment fields\r\n\r\n* Edit Person.java: isSamePerson considers all identity fields and test cases associated\r\n\r\n* Fix merge conflicts for test cases\r\n\r\n* Edit XmlAdaptedPay.java: fix equals\r\n\r\n* Fix AddressBookParserTest\r\n\r\n* Add newline characters to end of files\r\n\r\n* Edit XmlAdaptedPay/XmlUtilTest.java: fix test cases for payment\r\n\r\n* Fix variable type mismatch\r\n\r\n* Edit PersonUtil.java: include Subject import\r\n\r\n* Edit EditPersonDecriptorBuilder: add emptyPayment method\r\n\r\n* Edit Person.java: isSamePerson checks all identity fields\r\n\r\n* Fix xml files to include tutitionTiming and subject\r\n\r\n* Fix code style\r\n\r\n* Fix code style\r\n\r\n* Add MarkCommandSystemTest.java and MarkCommandTest.java\r\n\r\n* Add EraseSyllCommandTest.java, Fix MarkCommandTest.java: invalidSyllabusIndex is now correctly implemented\r\n\r\n* fix AddCommandSystemTest, AddCommandTest,\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix one line of checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add EraseSyllCommandSystemTest.java\r\n\r\n* Fix to style violations\r\n\r\n* Fixes to missing javadoc\r\n\r\n* Fix to EraseSyllCommandSystemTest.java: line with more than 120 characters\r\n\r\n* Fix XmlSerializableAddressBookTest\r\n\r\n* Fix HelpCommandSystemTest.java\r\n\r\n* Fix formatting of BrowserPanelTest.java to improve readability\r\n\r\n* Increase GuiRobot.java timeout to account for improved GUI\r\n\r\n* Fix BrowserPanelTest.java\r\n\r\n* Edit EditSyllCommandTest/EditSyllCommandParserTest.java: fix checkstyle\r\n\r\n* Fix minor checkstyle\r\n\r\n* Fix minor checkstyle\r\n\r\n* Fix minor checkstyle\r\n\r\n* Fix to display test\r\n\r\n* Fix AddCommandSystemTest\r\n\r\n* Add newline character to end of file\r\n\r\n* Fix checkstyle\r\n\r\n* Edit TypicalPerson.java: fix XmlSerializableAddressBookTest\r\n\r\n* fix checkstyle\r\n\r\n* Refactor AddressBook to TutorHelper\r\n\r\n* Fix some minor errors in refactoring\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add tests for GroupCommandParserTest\r\n\r\n* Update changes (#21)\r\n\r\n* Add CopySubCommandSystemTest.java\r\n\r\n* fix checkstyle\r\n\r\n* Change logic/commands/AppendSyllCommand.java: error checks for invalid subject added, Add logic/comands/AppendSyllCommandTest.java\r\n\r\n* Refactor AddressBook to TutorHelper\r\n\r\n* Fix some minor errors in refactoring\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Fix checkstyle\r\n\r\n* Add tests for GroupCommandParserTest\r\n\r\n* Add EditSyllCommandTest.java\r\n\r\n* Add GroupCommandTest and fix errors\r\n\r\n* Edit AppendSyllCommandTest.java: fix addressbook to tutorhelper\r\n\r\n* Edit CopySubCommandSystemTest.java: fix addressbook to tutorhelper\r\n\r\n* Edit GroupCommand.java: fix merge conlficts\r\n\r\n* Add EditSyllCommandSystemTest.java and fix EditCommandSystemTest.java\r\n\r\n* Edit EditSyllCommandSystemTest.java: remove unused imports\r\n\r\n* Edit GroupCommand.java: fix unused import\r\n\r\n* Edit imports to fix checkstyle\r\n\r\n* Fix checkstyle errors\r\n\r\n* Fix checkstyle errors: missing line at EOF\r\n\r\n* Fix checkstyle errors: remove unused imports","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/b97de0e5ed98d11547e597c9feb3d8de24857b2e"},{"sha":"8dac6385bfdba03c5705b6df81f0f923d8a0a3ed","author":{"name":"Joel Lee","email":"9a315c0c8b02f437ae938eb2dcea03500bf2aeb1@users.noreply.github.com"},"message":"Merge pull request #21 from CS2103-AY1819S1-W13-4/master\n\nUpdate fork","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/8dac6385bfdba03c5705b6df81f0f923d8a0a3ed"},{"sha":"a7bdcde85b7ee144c4c8eee84c9560fc4e1f373d","author":{"name":"ongyz","email":"76d010d5d553c82510249de7378ebacab71917df@gmail.com"},"message":"Update README.adoc: update appveyor","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/a7bdcde85b7ee144c4c8eee84c9560fc4e1f373d"},{"sha":"5f1a353f9e2faa8fe9b2fb47ac804aac8f655a7f","author":{"name":"andlimey","email":"e3f1ba8a1f233c48d8ba0a3251c971ec56c14b43@gmail.com"},"message":"Update README.adoc","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/5f1a353f9e2faa8fe9b2fb47ac804aac8f655a7f"},{"sha":"03288472729025ed087667396a3fd6fd6b806e8f","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Add AppendSyllCommandSystemTest.java","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/03288472729025ed087667396a3fd6fd6b806e8f"},{"sha":"f2825f6bfa12b0c8d181837a8de2a0c35dc7d7cc","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Fix to testutil/TypicalPersons.java and systemtests/TutorHelperSystemTest.java: remove redundant code","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/f2825f6bfa12b0c8d181837a8de2a0c35dc7d7cc"},{"sha":"ffdb004deda8eac96d43f8e1092e69ce807987db","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Fix systemtests/EditCommandSystemTest.java: remove redundant code","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/ffdb004deda8eac96d43f8e1092e69ce807987db"},{"sha":"7c33adac29a4701333970f9098054c324fcdcb4a","author":{"name":"Anthony Halim","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Merge branch 'master' into appendsyll-test","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/7c33adac29a4701333970f9098054c324fcdcb4a"},{"sha":"99d62c172b380effa236181ec072223a0349b759","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Fix to style violation","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/99d62c172b380effa236181ec072223a0349b759"},{"sha":"a2a486335317574c757f3e0a335fc60c0891be6e","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Merge branch 'appendsyll-test' of https://github.com/SusterDrifter/main into appendsyll-test","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/a2a486335317574c757f3e0a335fc60c0891be6e"},{"sha":"c6bbfacd1938b865959381bcc5bced236967ab6e","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Fix AppendSyllCommandSystemTest.java: fix to style violation","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/c6bbfacd1938b865959381bcc5bced236967ab6e"},{"sha":"75c9a4d3118a5cb32cf47947dbc53723baa6c3ac","author":{"name":"Anthony Halim","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Merge pull request #148 from ooihuiying/master\n\nAdd PayCommandSystemTest","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/75c9a4d3118a5cb32cf47947dbc53723baa6c3ac"},{"sha":"b36403368995ce528209051550011791fed0601f","author":{"name":"Anthony Halim","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Merge pull request #150 from ongyz/master\n\nUpdate README.adoc: update appveyor","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/b36403368995ce528209051550011791fed0601f"},{"sha":"ce1bd9c28d9e83a8bc1e7265ae153ea54c7a9cd5","author":{"name":"Anthony Halim","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Merge branch 'master' into appendsyll-test","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/ce1bd9c28d9e83a8bc1e7265ae153ea54c7a9cd5"},{"sha":"e24bb99e45d79f9d15a832d302d2136bb523b6d6","author":{"name":"andlimey","email":"e3f1ba8a1f233c48d8ba0a3251c971ec56c14b43@gmail.com"},"message":"Add GroupCommandSystemTest","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/e24bb99e45d79f9d15a832d302d2136bb523b6d6"},{"sha":"c7ce70df53d644d87c566ad7d375e0f2147fb2b6","author":{"name":"andlimey","email":"e3f1ba8a1f233c48d8ba0a3251c971ec56c14b43@gmail.com"},"message":"Fix checkstyle violations","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/c7ce70df53d644d87c566ad7d375e0f2147fb2b6"},{"sha":"20bf5da590d52f8515c15c5d98c60ae7084ec7f5","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Add SubjectsUtilTest","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/20bf5da590d52f8515c15c5d98c60ae7084ec7f5"},{"sha":"2a956d01f8c4b72d2d4731d5543f8908ef3d88ad","author":{"name":"Joel Lee","email":"ba800ce256e81923d838eccae3f989dfb8b0aad3@gmail.com"},"message":"Make Command Parsers inherit from Parser class","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/2a956d01f8c4b72d2d4731d5543f8908ef3d88ad"},{"sha":"f3dd0f4790d4567aef1d7fdd4f5b04cff9597e13","author":{"name":"SusterDrifter","email":"48037b2563b95f4eca506823f779dd0d11f86f2c@gmail.com"},"message":"Merge branch 'subjectutils-test' into appendsyll-test","distinct":true,"url":"https://api.github.com/repos/ooihuiying/main/commits/f3dd0f4790d4567aef1d7fdd4f5b04cff9597e13"}]}
{"actor":{"display_login":"ooihuiying"}}