-
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
Update Developer Guide: Use Case #15
Update Developer Guide: Use Case #15
Conversation
Use Case for 1. Done 2. View 3. Find 4. List 5. Exit
Addressbook -> TR4CKER Usecase resume -> ends
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.
Looks good and detailed, just some minor tweaks and standardisation and it should be good to merge!
docs/DeveloperGuide.md
Outdated
Use case ends. | ||
|
||
**Extensions** | ||
**Extentions** |
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.
Extensions is spelt inaccurately. Subsequently also. 😅
docs/DeveloperGuide.md
Outdated
|
||
* 3a1. TR4CKER shows an error message. | ||
|
||
Use case resumes at step 3. |
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.
I think use case should end here.
docs/DeveloperGuide.md
Outdated
* 3a. The given index is invalid. | ||
|
||
* 3a1. AddressBook shows an error message. | ||
* 3a1. TR4CKER shows an error message. | ||
|
||
Use case resumes at step 2. |
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.
I think use case should end here.
docs/DeveloperGuide.md
Outdated
|
||
* 2a. No tasks match the specified keyword(s). | ||
|
||
TR4CKER shows a 'no matches' error message. |
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.
I think indexing of 2a1 would be good.
docs/DeveloperGuide.md
Outdated
|
||
1. User requests to list tasks. | ||
2. TR4CKER shows a list of all tasks. | ||
3. Use case ends. |
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.
I think this line should not have indexing.
Extensions Typo
Codecov Report
@@ Coverage Diff @@
## master #15 +/- ##
=========================================
Coverage 72.31% 72.31%
Complexity 400 400
=========================================
Files 70 70
Lines 1228 1228
Branches 124 124
=========================================
Hits 888 888
Misses 308 308
Partials 32 32 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.
Looks good to merge. 👍🏽
Use Case for