-
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.1][T09-B3] ProgressChecker #46
base: master
Are you sure you want to change the base?
Changes from 35 commits
f9bd728
4bd5ab1
24c17ae
1610b0e
8cc5cca
6dc11a7
36fe7e9
f0957c3
3584b71
e0518bc
0754f81
c44b6da
5fdd135
f23e6d3
bc0980d
ed1ea42
ae78571
248cc75
18e56d5
46dcce6
e8586cd
f1c5529
db59bee
5bc8b2a
f054daa
8d74cb8
ca1dc7d
42d9e3a
f5d2473
68de537
cacf2bf
806ca97
5341412
2e8f99e
e5d561c
8333ea7
eff60cb
5904e2d
1609790
633a544
7b68cab
d68f915
a6f91fc
f97c0e9
33bbad9
d46e8db
6761637
365d2ef
1df215f
5b4a307
b830a00
18390a8
47f55d3
9a47dda
f9f7415
ce217bc
ed999c7
5ad04c8
dc409a8
dd3231e
f6b3d3d
e9764a7
50df3b2
9b012ed
1882f9c
c8323a2
a915f91
84ee2c5
55b91b4
c85e839
5fcf3a0
d6e0565
e173a01
0071e91
e708be4
5244764
26bbdca
3bea7fe
b741fb0
338ebe7
1751863
3a05f50
2f34f0e
f656178
061ee32
d5f33f3
7962718
7791eac
126511c
06d9415
c2b153f
97d3c44
26c3547
c8f2a4d
67757fe
b56570e
ad712bd
b24ac54
1c473b4
e777a3e
3f01b5d
9df2f97
1f1432c
2a12c31
2f43aa1
50ff7bc
30fadab
e9652b1
5411291
45b123c
565f598
b1fff1c
f3a1797
1d19a2c
5287588
df408a6
56db971
9b430bd
66166f0
3f102d4
77c25de
28fd147
075f76d
d685677
0013c16
621db04
30b64f1
5c996b0
9e3a66b
d4cee07
73b338f
641ecc0
772d63e
d8214f7
e11fed0
c76cf2f
d0dc4f6
7fcb46d
1904b8d
cb66253
4d4f21c
ba788d4
cd13ffd
436de55
cc80dc3
6148570
52417ea
f5d2517
9bcb079
609f16e
40ae3de
05e86cc
ee2d603
81b80d4
27c4bce
dde8ec8
863caad
591b696
4da2f4a
8e29e1f
6d08623
8c2fcb0
829fb28
552af99
a3fd411
ef70292
80e675d
777c13f
2b78703
1436f94
d23ee93
7137743
68cf753
ba1e60b
92ca327
81835be
f59e689
73747d6
c3be89a
7d54874
6abae80
c653dfc
51eb6ad
824fb3d
436c3a7
066cf0c
e4a45ee
4f245d5
be7e551
17c56d1
5e6e377
6357476
f17f66b
03de976
581434b
c0d94ad
e18ce5e
7489f2a
5a09d97
3168ded
404fd74
ccf9d9a
d3d899f
2b0a5ad
058436e
2d04159
b958a69
909e2ac
2382216
93af57c
131a46b
453111c
ac2b099
32655ad
9c26e87
40d57b6
4c15c7d
a5cf1bc
f93abd6
edce9f9
7a2e86e
b0d0ac4
de851ee
49466c3
e48ed56
470bbd5
1a7cb00
1f640a8
861e6a9
b670b72
0057c37
a237f69
0dcdd7e
c225801
7070bfc
a4521c0
23bc50b
d11da5f
a529fe3
524dfd9
5b7b356
3299bbe
e1103bb
833cb6f
e433364
e229fba
a8dcc76
e9d69de
4fbbd50
2e88b81
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,9 +10,9 @@ 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-B3/addressbook-level4/tree/master | ||
|
||
By: `Team SE-EDU` Since: `Jun 2016` Licence: `MIT` | ||
By: `Team T09-B3` Since: `Jun 2016` Licence: `MIT` | ||
|
||
== Setting up | ||
|
||
|
@@ -782,13 +782,23 @@ See this https://github.com/se-edu/addressbook-level4/pull/599[PR] for the step- | |
|
||
*Target user profile*: | ||
|
||
* is taking CS2103/CS2103T in NUS | ||
* wants to keep track on his/her learning progress | ||
* is willing to have good team management | ||
* has a need to manage a significant number of contacts | ||
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. Are you sure this point is still relevant? |
||
* prefer desktop apps over other types | ||
* prefers desktop apps over other types | ||
* can type fast | ||
* prefers typing over mouse input | ||
* is reasonably comfortable using CLI apps | ||
|
||
*Value proposition*: manage contacts faster than a typical mouse/GUI driven app | ||
*Value proposition*: | ||
|
||
* never miss LOs hidden in the nested collapsible list | ||
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 there any? If so, please submit an issue to the its github repo. |
||
* keep track of your own progress on a week by week basis | ||
* keep track of your teammates' completion of TO-DO list(at most 20 tasks per day) and learning progress (providing proof for peer evaluation) | ||
* view the teammates' weekly plan in phase B (to know the upcoming updates even before they send any pull requests to github repo) | ||
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. As I said in the tutorial, it might overlap with the issue tracker functionality. |
||
* save your answers(no more than 200 words) for the exercises (as proof of completion and for future revision) | ||
* easily search for information based on key words (which is hard to achieve in the module website with nested collapsible lists) | ||
|
||
[appendix] | ||
== User Stories | ||
|
@@ -800,11 +810,35 @@ 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 | | ||
|`* * *` |new user |register and account with NUS net ID |join the module community | ||
|
||
|`* * *` |new user |fill in my full name and phone number (strictly 8 numbers) |provide necessary information for the platform maintenance | ||
|
||
|`* * *` |new user |fill in optional fiels such as faculty, year of study etc |help my classmates know be better | ||
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. Since this is optional, shouldn't it has two stars? |
||
|
||
|`* * *` |user |add contacts |have more classmates to communicate with | ||
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. How are you going to implement this? chatroom? If so, there is no user story that supports this communication feature. |
||
|
||
|`* * *` |user |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 | ||
|`* * *` |user |find a person by NUS NET ID |locate details of persons without having to go through the entire list | ||
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. Do you remember your classmate NUSNET ID? Also, instead of using "person" may be "classmate" is better. |
||
|
||
|`* * *` |user |search information in our module website based on keywords |nevigate and find the information I need quickly | ||
|
||
|`* * *` |user |view my TO-DO list |know all the my tasks easily | ||
|
||
|`* * *` |user |tick an item in the TO-DO list upon completion |focus on the tasks I have not done | ||
|
||
|`* * *` |user |save my answers of the exercises |show to tutor as proof of my learning outcome and revise before exams | ||
|
||
|`* * *` |user |update information of certain field(s) |keep my information up-to-date | ||
|
||
|`* * *` |user |see the timeline showing the learning progress of me and my teammates |make sure everyone is on track | ||
|
||
|`* * *` |user |see the weekly plan of my teammates (phase B) |to know the upcoming updates even before they send any pull requests to github repo | ||
|
||
|`* * *` |user |post my weekly plan (phase B) |to inform my teammates of upcoming updates even before I send any pull requests to github repo | ||
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. Seems like there are too many three stars features, re-consider if those features really critical for this app. |
||
|
||
|`* *` |user |upload a profile picture (not larger than 3MB) |help others to recognize me | ||
|
||
|`* *` |user |hide <<private-contact-detail,private contact details>> by default |minimize chance of someone else seeing them by accident | ||
|
||
|
@@ -844,6 +878,92 @@ Use case ends. | |
+ | ||
Use case resumes at step 2. | ||
|
||
[discrete] | ||
=== Use case: Add a to-do | ||
|
||
*MSS* | ||
|
||
1. User requests to view tasks in a specific week | ||
2. AddressBook shows tasks in the specified week | ||
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. I believe you won't call this AddressBook, right? |
||
3. User requests to add a new task in a specific week | ||
4. AddressBook adds a new task to the existing list | ||
+ | ||
Use case ends. | ||
|
||
*Extensions* | ||
|
||
* 1a. Specified week does not exist. | ||
+ | ||
[none] | ||
** 1a1. AddressBook shows an error message. | ||
+ | ||
Use case resumes at step 2. | ||
|
||
[discrete] | ||
=== Use case: Mark a to-do as complete | ||
|
||
*MSS* | ||
|
||
1. User requests to view tasks in a specific week | ||
2. AddressBook shows tasks in the specified week | ||
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. AddressBook? Use your app name, instead. |
||
3. User requests to complete a task in a specific week | ||
4. AddressBook marks specified task as complete | ||
+ | ||
Use case ends. | ||
|
||
*Extensions* | ||
|
||
* 1a. Specified week does not exist. | ||
+ | ||
[none] | ||
** 1a1. AddressBook shows an error message. | ||
+ | ||
Use case resumes at step 2. | ||
|
||
[none] | ||
* 2a. The tasks list is empty. | ||
+ | ||
Use case resumes at step 2. | ||
|
||
* 3a. The specified task does not exist | ||
+ | ||
[none] | ||
** 3a1. AddressBook shows an error message. | ||
+ | ||
Use case ends. | ||
|
||
[discrete] | ||
=== Use case: Answer a question and save | ||
|
||
*MSS* | ||
|
||
1. User requests to view a specific week | ||
2. AddressBook shows content of the specified week | ||
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. "content" is unclear.. Does it include the to-do list? I guess not. |
||
3. User requests to key in and save an answer to a question | ||
4. AddressBook takes in input and saves | ||
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. May be these steps can be split into more specific steps. |
||
+ | ||
Use case ends. | ||
|
||
*Extensions* | ||
|
||
* 1a. Specified week does not exist. | ||
+ | ||
[none] | ||
** 1a1. AddressBook shows an error message. | ||
+ | ||
Use case resumes at step 2. | ||
|
||
[none] | ||
* 2a. There is no content to be shown. | ||
+ | ||
Use case ends. | ||
|
||
* 3a. The question index does not exist. | ||
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. How come suddenly there is an "index" mentioned here? |
||
[none] | ||
** 3a1. AddressBook shows an error message. | ||
+ | ||
Use case ends. | ||
|
||
_{More to be added}_ | ||
|
||
[appendix] | ||
|
@@ -852,6 +972,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. | ||
. The product may need 3-5 minutes to build up for the first time. | ||
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. Remove doubtful words such as "may", "might" and so on. |
||
. The data cannot be retrieved from outside | ||
. User need to log in with their unique ID and corresponding password | ||
. User can only view the information of their teammates | ||
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. I think the last three points are not NFRs. |
||
|
||
_{More to be added}_ | ||
|
||
|
@@ -864,6 +988,18 @@ Windows, Linux, Unix, OS-X | |
[[private-contact-detail]] Private contact detail:: | ||
A contact detail that is not meant to be shared with others | ||
|
||
[[GUI]] GUI:: | ||
Graphical User Interface | ||
|
||
[[sequence-diagram]] Sequence Diagram:: | ||
A sequence diagram shows object interactions shown in time sequence. | ||
|
||
[[Build-Automation]] Build Automation:: | ||
Build automation is the process of automating the creation of a software build and the associated processes including: compiling computer source code into binary code, packaging binary code, and running automated tests. | ||
|
||
[[Gradle]] Gradle:: | ||
Gradle is an open-source build automation system. | ||
|
||
[appendix] | ||
== Product Survey | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
#Mon Mar 21 13:28:19 SGT 2016 | ||
#Wed Mar 07 06:08:43 CST 2018 | ||
distributionBase=GRADLE_USER_HOME | ||
distributionPath=wrapper/dists | ||
zipStoreBase=GRADLE_USER_HOME | ||
zipStorePath=wrapper/dists | ||
distributionUrl=https\://services.gradle.org/distributions/gradle-2.12-all.zip | ||
distributionUrl=https\://services.gradle.org/distributions/gradle-2.12-bin.zip |
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.
Good job for updating the badges.