-
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
Fix bugs with misuse of sample visits and update UG #212
Fix bugs with misuse of sample visits and update UG #212
Conversation
Bug: It is possible for MainApp to initialise using sample visits without sample locations and persons. This is a bug since visitbook would be having visits where the persons and locations involved are not in personbook and locationbook. Solution: If any of the location, person and visit data is empty, use all of sample visits, persons and locations.
…into v1.3/Refactor # Conflicts: # src/main/java/seedu/address/ui/HelpWindow.java
Codecov Report
@@ Coverage Diff @@
## master #212 +/- ##
============================================
+ Coverage 77.75% 78.04% +0.28%
Complexity 996 996
============================================
Files 125 125
Lines 2985 2974 -11
Branches 399 397 -2
============================================
Hits 2321 2321
+ Misses 535 524 -11
Partials 129 129
Continue to review full report at Codecov.
|
…into v1.3/Refactor # Conflicts: # 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.
LGTM
Summary command table in UG is deleted. It can be added back with the correct commands in next iteration. |
This is a draft Pull Request