-
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 use cases in DG #262
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.
lgtm, needs minor changes
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! i commented on some minor things you might have missed out on!
docs/DeveloperGuide.md
Outdated
|
||
**Extensions** | ||
|
||
* 1a. User leaves out parameters. |
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.
Need to be more specific about which parameters are left out
docs/DeveloperGuide.md
Outdated
|
||
**Extensions** | ||
|
||
* 1a. User leaves out parameters. |
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.
Need to be more specific about which parameters are left out
docs/DeveloperGuide.md
Outdated
**MSS** | ||
|
||
1. User requests to add a given remark to an employee with a given employee id. | ||
2. HouR adds the remark to the list of remarks associated with the employee with the given employee id. |
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.
The use case stories should alternate between the system and the user
docs/DeveloperGuide.md
Outdated
|
||
**Extensions** | ||
|
||
* 1a. User leaves out parameters. |
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.
Need to be more specific about which parameters
docs/DeveloperGuide.md
Outdated
|
||
1. User requests to delete a given remark from an employee with a given employee id. | ||
2. HouR deletes the remark from the list of remarks associated with the employee with the given employee id. | ||
3. HouR shows the list of remarks associated with the employee with the given employee id. |
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.
MSS should alternate between user and system
…4S1-CS2103T-W12-1/tp into v1.4-DeveloperGuide-Update
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.