-
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
Filtering by event #147
Filtering by event #147
Conversation
…into filter-event
…into filter-event
…nto filter-event
…into filter-event
Codecov Report
@@ Coverage Diff @@
## master #147 +/- ##
============================================
- Coverage 67.38% 63.70% -3.69%
- Complexity 562 616 +54
============================================
Files 99 110 +11
Lines 1984 2229 +245
Branches 226 261 +35
============================================
+ Hits 1337 1420 +83
- Misses 542 707 +165
+ Partials 105 102 -3 Continue to review full report at Codecov.
|
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 small nit pick
src/main/java/seedu/address/logic/commands/FilterEventCommand.java
Outdated
Show resolved
Hide resolved
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
Filtering residents by events. #135
E.g.
filter-event 1
displays all residents attending the event 1 on the event listTest cases for
filter-event
will be added in the future