-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update User Guide and Developer Guide #127
Conversation
`delete 1` -> `contact delete 1`
`delete 1` -> `contact delete 1`
Rename JsonSerializablePerson to JsonAdaptedPerson. Add JsonAdaptedReminder and JsonAdaptedMeeting
Codecov Report
@@ Coverage Diff @@
## master #127 +/- ##
=========================================
Coverage 75.03% 75.03%
Complexity 1147 1147
=========================================
Files 161 161
Lines 3633 3633
Branches 496 496
=========================================
Hits 2726 2726
Misses 734 734
Partials 173 173 Continue to review full report at Codecov.
|
…into docs # Conflicts: # docs/DeveloperGuide.md # docs/UserGuide.md
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.
Thanks @sebastiantoh for updating the docs! LGTM!
📔 This PR contains work from multiple team members and is not solely my (Sebastian) work
Changes:
Introduction
Design
delete 1
->contact delete 1
Implementation
Implementation
section to include some meeting implementation detailsImplementation
section to include some sale implementation detailsFuture Improvements:
Logic
is inaccurate, specifically the part for the Parser since we made some amendments / refactoringStonksBookParser
. I'm not sure if we should renameAddressBookParser
toStonksBookParser
, and then update the UML diagrams[Proposed] Undo/redo feature
and[Proposed] Data archiving
Appendix: Instructions for manual testing