[T7][T15-C2]#92
[T7][T15-C2]#92zhengyu92 wants to merge 49 commits intonus-cs2103-AY1617S1:masterfrom CS2103AUG2016-T15-C2:master
Conversation
Added items to "Non-Functional Requirements"
Edited User Guide
Edited User Guide from skeleton to our project
Added niveetha image
K1ang
left a comment
K1ang
left a comment
There was a problem hiding this comment.
@zhengyu92 @chankyuoh @Niveetha @takukou
Good job in doing up this userguide!
Most of your commands seems user friendly.
There are some parts that can still be thought through more thoroughly to have a better design for your application.
| 2. Scheduler shows a list of items or the item | ||
| 3. User calls the edit function | ||
| 4. User types in index of item and inputs new values for the item | ||
| 5. Scheduler displays updated list of items <br> |
There was a problem hiding this comment.
Is displaying the list of items necessary? Or just the details of the updated item will be better.
| 2a. The list is empty | ||
|
|
||
| > 2a1. Scheduler displays empty list <br> | ||
|
|
There was a problem hiding this comment.
More thinking through is required for this function. What does it mean by tracking items which are done and which are yet to be done? You can consider functions to allow the user to mark a task to be completed. And how should a user list completed/uncompleted task?
docs/UserGuide.md
Outdated
| * **`exit`** : exits the app | ||
| * **`list`** : lists all tasks | ||
| * **`add`**` `add Do Homework d/ 19/02/12 t/13:43 n/ cs2103 homework` : | ||
| adds a task named `Do Homework` to the Task List. |
There was a problem hiding this comment.
Inconsistent with the details of add command in the features section.
| **MSS** | ||
|
|
||
| 1. User calls the find function | ||
| 2. Scheduler displays the items that are word related with no case sensitive |
There was a problem hiding this comment.
Seems ambiguous here. What is word related?
| Clears all entries from the address book.<br> | ||
| Format: `clear` | ||
| * `find Tutorial`<br> | ||
| Returns `CS2103 Tutorial` and words related to `tutorial` |
There was a problem hiding this comment.
What does related means? Can be more specific for this.
| * `find Tutorial`<br> | ||
| Returns `CS2103 Tutorial` and words related to `tutorial` | ||
| * `find CS Assignment Errand`<br> | ||
| Returns Any item having names `CS`, `Assignment`, or `Errand` |
There was a problem hiding this comment.
Does this return "any item" that matches either of the multiple keywords (1 item only) or all items that matches with either of the keywords?
| #### [Henderson](http://github.com/takukou) | ||
| Role: Developer <br> | ||
| Responsibilities: Storage | ||
|
|
There was a problem hiding this comment.
Check out the handbook on some possible roles and responsibilities other than developer. :)
| #### Saving the data: `path` | ||
| Scheduler data are saved in the hard disk automatically after any command that changes the data. File path can be manually changed to a user-specified location.<br> | ||
| Format: `path FILE_PATH` | ||
|
|
There was a problem hiding this comment.
Saving the data and setting a new file path should be separated into different components.
|
Some comments added. Please close the PR after reading comments. |
# Conflicts: # docs/DeveloperGuide.md # docs/UserGuide.md
* commandChange: Change all relevant 'address' to 'date' Rename all 'email' to 'endDateTime' Rename all 'phone' to 'startDateTime' Change all 'person' to 'task' Delete SelectCommand.java and related codes # Conflicts: # docs/DeveloperGuide.md # docs/UserGuide.md
* 'master' of https://github.com/CS2103AUG2016-T15-C2/main: # Conflicts: # docs/DeveloperGuide.md # docs/UserGuide.md
5 participants
Work in progress