-
Notifications
You must be signed in to change notification settings - Fork 263
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
[CS2103T-W16-01] B B Productive #22
base: master
Are you sure you want to change the base?
[CS2103T-W16-01] B B Productive #22
Conversation
User Stories:
|
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.
Some formatting inconsistencies. But overall content looks good!
docs/DeveloperGuide.adoc
Outdated
+ | ||
Use case resumes at step 2. | ||
|
||
_{More to be added}_ | ||
=== Use case: Run task with Pomodoro |
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.
Additional C.1. when compiled. Small formatting inconsistencies.
docs/DeveloperGuide.adoc
Outdated
|
||
*MSS* | ||
|
||
1. User requests to run task with Pomodoro |
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.
no full stops!
docs/DeveloperGuide.adoc
Outdated
1. User requests to list persons | ||
2. AddressBook shows a list of persons | ||
1. User requests to list tasks | ||
2. BBProductive shows a list of tasks | ||
3. User requests to delete a specific person in the list |
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 this supposed to be a task?
docs/DeveloperGuide.adoc
Outdated
*MSS* | ||
|
||
1. User requests to run task with Pomodoro | ||
2. User proceeds to do task |
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.
- System starts Pomodoro timer?
docs/DeveloperGuide.adoc
Outdated
** 2a1. BBProductive shows an error message. | ||
Use case ends. | ||
|
||
* 3a. User has not completed the task. |
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.
Shouldn't this be 4a?
docs/DeveloperGuide.adoc
Outdated
+ | ||
Use case resumes at step 3. | ||
|
||
* 3b. User has not completed the task but wants to take a break from completing the task. |
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.
4b?
docs/DeveloperGuide.adoc
Outdated
[none] | ||
** 3a1. User exits the Pomodoro without marking the task as done. | ||
+ | ||
Use case ends. | ||
|
||
[appendix] | ||
== Non Functional Requirements | ||
|
||
. Should work on any <<mainstream-os,mainstream OS>> as long as it has Java `11` or above installed. | ||
. Should be able to hold up to 1000 persons without a noticeable sluggishness in performance for typical usage. |
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.
Did you mean 1000 tasks?
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.
I couldn't find any significant updates (especially UML Diagrams) to give comments/suggestions on. Please update your DG soon!
* Updates LogicClassDiagram image file. * Fixes final. Co-authored-by: Hardy Shein <>
…hange Changed fonts to suit coverpage
Finalized UG for 2101
Merge to master for updated CS2101 pdf generation No code modified as per guidelines in https://nus-cs2103-ay1920s2.github.io/website/schedule/week13/project.html
2dcc587
to
0ed45a9
Compare
@Hardoh
@FyonnOh
@artlee06
@Permas-Teo