-
Notifications
You must be signed in to change notification settings - Fork 5
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
Rename AddressBook and Person #55
Rename AddressBook and Person #55
Conversation
…uan/tp into ypinhsuan-rename-addressbook-person # Conflicts: # src/main/java/seedu/address/logic/commands/AddCommand.java # src/main/java/seedu/address/logic/commands/EditCommand.java # src/main/java/seedu/address/logic/parser/AddCommandParser.java # src/main/java/seedu/address/logic/parser/EditCommandParser.java # src/main/java/seedu/address/logic/parser/ParserUtil.java # src/main/java/seedu/address/model/student/Address.java # src/main/java/seedu/address/model/student/Student.java # src/main/java/seedu/address/model/util/SampleDataUtil.java # src/main/java/seedu/address/storage/JsonAdaptedStudent.java # src/main/java/seedu/address/ui/StudentCard.java # src/test/data/JsonSerializableTutorsPetTest/duplicateStudentTutorsPet.json # src/test/data/JsonSerializableTutorsPetTest/typicalStudentsTutorsPet.json # src/test/java/seedu/address/logic/LogicManagerTest.java # src/test/java/seedu/address/logic/commands/CommandTestUtil.java # src/test/java/seedu/address/logic/commands/EditStudentDescriptorTest.java # src/test/java/seedu/address/logic/parser/AddCommandParserTest.java # src/test/java/seedu/address/logic/parser/EditCommandParserTest.java # src/test/java/seedu/address/logic/parser/ParserUtilTest.java # src/test/java/seedu/address/model/AddressBookTest.java # src/test/java/seedu/address/model/person/UniquePersonListTest.java # src/test/java/seedu/address/model/student/AddressTest.java # src/test/java/seedu/address/model/student/NameContainsKeywordsPredicateTest.java # src/test/java/seedu/address/model/student/StudentTest.java # src/test/java/seedu/address/storage/JsonAdaptedStudentTest.java # src/test/java/seedu/address/testutil/EditPersonDescriptorBuilder.java # src/test/java/seedu/address/testutil/PersonBuilder.java # src/test/java/seedu/address/testutil/StudentUtil.java # src/test/java/seedu/address/testutil/TypicalPersons.java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work! LGTM.
Might have to reconsider the wording of certain comments down the road because previously address book could refer to both the name and the purpose of the product, but Tutor's Pet can only refer to the name of our new product.
But to prevent this PR from being a blocker, I think we can roll with the changes made here first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Perhaps we can change the use of 'application' to something like 'manager' or 'tutors book' (to replace address book)?
No description provided.