-
Notifications
You must be signed in to change notification settings - Fork 69
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
[v1.5][T09-B2] CollegeZone #47
base: master
Are you sure you want to change the base?
Changes from 52 commits
703cd54
450bf5f
22536a8
621c00f
c1262a2
775fb29
eb6f298
766655f
3488202
2f983bd
13dec3b
9e1e439
67172ae
af428a0
ff7d0d3
f5d4a09
1186e43
e189adc
b9cf011
1cc1c1f
6915523
97df1f8
38b3de6
5193f3d
e1cc3a3
69eaf58
36fb222
2e3da1f
96cc55d
362e0b7
9761466
1960eb6
6bf5df2
cdef5f7
4e1541d
7f011ec
37cf851
3e43a75
fd4c4c6
b983eb1
9bebf93
d2f9a7a
516eeef
38dcd2c
afd0f0a
db6a3a9
4fe1a8e
0d7ef43
734e3b4
57ece24
c5f8cea
5b696f9
1b85d39
962000d
c5e1372
cd3d814
12e89a4
9d5c495
9f1a5f8
25c4024
238c87d
a4d1b2c
78696fd
cfb4971
fe8b58a
4c6a264
13757d8
e18832f
cc197cc
b852429
647cd13
5eb23f1
e8bac6b
a2a7912
6b94050
305a467
edeeced
f210c81
2245a31
aee3303
7e1c4ca
929bbd5
caac39b
990790f
e808c1f
04efdfa
96caf93
2d9765f
c850cdf
4794b8a
33c81e2
0937ea9
52a0abf
16ea065
c811429
a0bb682
c2fa8aa
04d9aa3
58693a6
160935c
a1f01b9
8654e19
93ece59
2d98925
a873b5e
6306ff6
9335c56
f94be60
57838b6
50f32e0
e51fde2
25b7d04
9831f88
54219cd
25191d0
3447b80
f6ee080
c4b96c3
cfb54d5
ed58cb1
39a2abd
1a21545
6f25a65
e25f8b1
90c9723
b845639
5f89554
66590be
2ec1c98
1708988
c2c8c6c
7647cc2
14bbbc9
339b370
c698780
f4fbab8
26e93bf
29af6a8
d1dddfc
d7afea8
bab3504
d1c8aac
91e0a85
1751065
9c55aae
ab457f8
2a4acd1
d8e3d6a
9a6f52b
965e849
4e159e1
fd2f670
51a8ef8
173ad5b
1d8e53a
452956b
03e09ff
35b6a52
59e4638
2f9bf1c
b6ced3c
85dfb46
51c0a32
7302214
8c7296c
2a89138
574be32
7e84ff7
893dce5
dadc438
eb94300
d79fb1c
319a725
657b111
4d7a125
453178b
19ba82a
4931ab6
d6e901c
9e6c1f5
5cea0ed
7e467d0
49bcd42
9d3bc8c
febdb49
3ef460a
7ca9d20
7c84114
ff644ac
1aec6ce
e35c515
cf79170
74649c4
114ab16
8c5bfde
bb1b7d0
c444e8a
6e10556
1a6ead0
f409046
7ecaf02
5327e84
a83c69f
e73b282
2b192c9
19f88a7
1a7320b
484437b
5122f1a
60965d4
8d5a088
3f7fead
3f5cfbd
2f432b9
3f64ca1
ddb4c6e
1c76711
55d6f46
5b7e395
879325e
c4dab30
d426fbf
964a16d
1913e22
8aedd70
4a17c6b
daef627
6e3dfff
1bfa06a
861fec2
1cc7cd5
2ac8d9b
8ba713c
de6975d
1d6deb5
089c3d0
ea1c1a1
3d0fd08
24467d5
59dc583
d251738
a0b864e
9f25394
864cfcd
ffd6a7b
4a9f755
c180847
6124dd0
942df1f
ba882f2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,7 +10,7 @@ ifdef::env-github[] | |
:tip-caption: :bulb: | ||
:note-caption: :information_source: | ||
endif::[] | ||
:repoURL: https://github.com/se-edu/addressbook-level4/tree/master | ||
:repoURL: https://github.com/CS2103JAN2018-T09-B2/main | ||
|
||
By: `Team SE-EDU` Since: `Jun 2016` Licence: `MIT` | ||
|
||
|
@@ -782,6 +782,7 @@ See this https://github.com/se-edu/addressbook-level4/pull/599[PR] for the step- | |
|
||
*Target user profile*: | ||
|
||
* NUS Students living in RC | ||
* has a need to manage a significant number of contacts | ||
* prefer desktop apps over other types | ||
* can type fast | ||
|
@@ -800,33 +801,60 @@ Priorities: High (must have) - `* * \*`, Medium (nice to have) - `* \*`, Low (un | |
|Priority |As a ... |I want to ... |So that I can... | ||
|`* * *` |new user |see usage instructions |refer to instructions when I forget how to use the App | ||
|
||
|`* * *` |user |add a new person | | ||
|`* * *` |RC student |add a new person | | ||
|
||
|`* * *` |user |delete a person |remove entries that I no longer need | ||
|`* * *` |RC student |delete a person |remove entries that I no longer need | ||
|
||
|`* * *` |user |find a person by name |locate details of persons without having to go through the entire list | ||
|`* * *` |RC student |find a person by name |locate details of persons without having to go through the entire list | ||
|
||
|`* * *` |RC student |find a person by tags |locate a particular group of people without having to go through the entire list | ||
|
||
|`* * *` |RC student |edit a detail I added | | ||
|
||
|`* * *` |user |set a level of friendship with a specific person |maintain my friendships depending on a priority system set by myself | ||
|
||
|`* * *` |user |edit details of my contacts |stay updated with the current details of my friends | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Here |
||
|
||
|`* * *` |forgetful RC student |to add persistent reminders |periodically remind myself to do something. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Second column inconsistent with the other user stories? |
||
|
||
|`* * *` |forgetful RC student |add other RC friends name, birthday, hall CCAs and tags into CollegeZone | | ||
|
||
|`* *` |careless RC student |undo a command I entered |undo a wrong command that I entered | ||
|
||
|`* *` |careless RC student |redo a command I entered |redo when I want to undo my "undo" command | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No need to repeat the features that current version of addressbook already contains. |
||
|
||
|`* *` |user |note down appointments with my friends in a calendar |make my schedule schedule more organised | ||
|
||
|`* *` |user |write down a short reflection of how an appointment went |remember precious moments easier in the future | ||
|
||
|`* *` |user |list down all past appointments with a particular friend |reminisce past memories with a particular friend | ||
|
||
|`* *` |user |hide <<private-contact-detail,private contact details>> by default |minimize chance of someone else seeing them by accident | ||
|
||
|`* *` |RC student | be reminded on when my campus fees are due | pay it on time | ||
|
||
|`*` |user with many persons in the address book |sort persons by name |locate a person easily | ||
|
||
|`*` |user with many persons with the same in the address book |set a display picture of each contact|differentiate persons with the same name | ||
|
||
|======================================================================= | ||
|
||
_{More to be added}_ | ||
|
||
[appendix] | ||
== Use Cases | ||
|
||
(For all use cases below, the *System* is the `AddressBook` and the *Actor* is the `user`, unless specified otherwise) | ||
(For all use cases below, the *System* is the `CollegeZone` and the *Actor* is the `user`, unless specified otherwise) | ||
|
||
[discrete] | ||
=== Use case: Delete person | ||
|
||
*MSS* | ||
|
||
1. User requests to list persons | ||
2. AddressBook shows a list of persons | ||
2. CollegeZone shows a list of persons | ||
3. User requests to delete a specific person in the list | ||
4. AddressBook deletes the person | ||
4. CollegeZone deletes the person | ||
+ | ||
Use case ends. | ||
|
||
|
@@ -840,7 +868,36 @@ Use case ends. | |
* 3a. The given index is invalid. | ||
+ | ||
[none] | ||
** 3a1. AddressBook shows an error message. | ||
** 3a1. CollegeZone shows an error message. | ||
+ | ||
Use case resumes at step 2. | ||
|
||
[discrete] | ||
=== Use case: Edit person | ||
1. User requests to list persons | ||
2. CollegeZone shows a list of persons | ||
3. User requests to edit a detail or multiple details of a person in the list | ||
4. CollegeZone edits the detail or details of the person | ||
+ | ||
Use case ends. | ||
|
||
*Extensions* | ||
|
||
[none] | ||
* 2a. The list is empty. | ||
+ | ||
Use case ends. | ||
|
||
* 3a. The given index is invalid. | ||
+ | ||
[none] | ||
** 3a1. CollegeZone shows an error message. | ||
+ | ||
|
||
* 3b. The given detail format is invalid. | ||
+ | ||
[none] | ||
** 3b1. CollegeZone shows an error message. | ||
+ | ||
Use case resumes at step 2. | ||
|
||
|
@@ -852,6 +909,10 @@ _{More to be added}_ | |
. Should work on any <<mainstream-os,mainstream OS>> as long as it has Java `1.8.0_60` or higher installed. | ||
. Should be able to hold up to 1000 persons without a noticeable sluggishness in performance for typical usage. | ||
. 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. | ||
. Should be able to deal with invalid command inputs. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this an NFR or a functional requirement? |
||
. Should be intuitive to use for users who are not tech-savvy. | ||
. The system should respond within 2 seconds. | ||
. Should work on 32-bit and 64-bit environment. | ||
|
||
_{More to be added}_ | ||
|
||
|
@@ -864,6 +925,9 @@ Windows, Linux, Unix, OS-X | |
[[private-contact-detail]] Private contact detail:: | ||
A contact detail that is not meant to be shared with others | ||
|
||
[[RC]] Residential College:: | ||
A campus living area at NUS U-Town for NUS undergraduate students, which includes Cinnamon College (USP), College of Alice & Peter Tan, Residential College 4 and Tembusu College | ||
|
||
[appendix] | ||
== Product Survey | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -51,27 +51,51 @@ public Command parseCommand(String userInput) throws ParseException { | |
case AddCommand.COMMAND_WORD: | ||
return new AddCommandParser().parse(arguments); | ||
|
||
case AddCommand.COMMAND_ALIAS: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You can actually combine the command word and command alias together for the same command to make it clearer |
||
return new AddCommandParser().parse(arguments); | ||
|
||
case EditCommand.COMMAND_WORD: | ||
return new EditCommandParser().parse(arguments); | ||
|
||
case EditCommand.COMMAND_ALIAS: | ||
return new EditCommandParser().parse(arguments); | ||
|
||
case SelectCommand.COMMAND_WORD: | ||
return new SelectCommandParser().parse(arguments); | ||
|
||
case SelectCommand.COMMAND_ALIAS: | ||
return new SelectCommandParser().parse(arguments); | ||
|
||
case DeleteCommand.COMMAND_WORD: | ||
return new DeleteCommandParser().parse(arguments); | ||
|
||
case DeleteCommand.COMMAND_ALIAS: | ||
return new DeleteCommandParser().parse(arguments); | ||
|
||
case ClearCommand.COMMAND_WORD: | ||
return new ClearCommand(); | ||
|
||
case ClearCommand.COMMAND_ALIAS: | ||
return new ClearCommand(); | ||
|
||
case FindCommand.COMMAND_WORD: | ||
return new FindCommandParser().parse(arguments); | ||
|
||
case FindCommand.COMMAND_ALIAS: | ||
return new FindCommandParser().parse(arguments); | ||
|
||
case ListCommand.COMMAND_WORD: | ||
return new ListCommand(); | ||
|
||
case ListCommand.COMMAND_ALIAS: | ||
return new ListCommand(); | ||
|
||
case HistoryCommand.COMMAND_WORD: | ||
return new HistoryCommand(); | ||
|
||
case HistoryCommand.COMMAND_ALIAS: | ||
return new HistoryCommand(); | ||
|
||
case ExitCommand.COMMAND_WORD: | ||
return new ExitCommand(); | ||
|
||
|
@@ -81,9 +105,15 @@ public Command parseCommand(String userInput) throws ParseException { | |
case UndoCommand.COMMAND_WORD: | ||
return new UndoCommand(); | ||
|
||
case UndoCommand.COMMAND_ALIAS: | ||
return new UndoCommand(); | ||
|
||
case RedoCommand.COMMAND_WORD: | ||
return new RedoCommand(); | ||
|
||
case RedoCommand.COMMAND_ALIAS: | ||
return new RedoCommand(); | ||
|
||
default: | ||
throw new ParseException(MESSAGE_UNKNOWN_COMMAND); | ||
} | ||
|
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.
Is it better to write as
Residential College (RC)
since other developers might not know whatRC
stands for?