-
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
Add use case, reduce NFR in DG #214
Conversation
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.
A few changes may be required
1. A user with above average typing speed for regular English text (i.e. not code, not system admin commands) | ||
should be able to accomplish most of the tasks faster using commands than using the mouse. | ||
1. The codebase should be well-structured and well-documented to facilitate future maintenance and enhancements. | ||
1. The application should only support a single user. | ||
1. The data should be stored locally and should be in a human editable text file. | ||
1. The software should work without requiring an installer. | ||
1. The software should not depend on any specific remote server. | ||
1. The product should be available as a single JAR file of size 100MB or below. | ||
1. The product should process a user input command within 2 second. |
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.
numbering here is incorrect
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.
docs/DeveloperGuide.md
Outdated
@@ -523,13 +523,13 @@ specified otherwise) | |||
* 2a. The given index is invalid. | |||
* 2a2. AddressBook shows an error message. | |||
|
|||
Use case resumes at step 2. | |||
Use case ends. | |||
|
|||
**Use case: `US04 - Edit a patient records`** |
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.
Should be UC instead of US
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.
ok
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
No description provided.