Skip to content
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

[W10-2] PocketProject #26

Open
wants to merge 1,355 commits into
base: master
Choose a base branch
from

Conversation

dehui333
Copy link

** A more sophisticated GUI that includes a list panel and an in-built Browser.
** More test cases, including automated GUI testing.
** Support for _Build Automation_ using Gradle and for _Continuous Integration_ using Travis CI.
* PocketProject (Project Manager)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the UI mock up is quite blurry, please provide a clearer one

AddressBook - Level 4 was developed by the https://se-edu.github.io/docs/Team.html[se-edu] team. +
_{The dummy content given below serves as a placeholder to be used by future forks of the project.}_ +
{empty} +
PocketProject was developed by the https://github.com/cs2103-ay1819s2-w10-2[W10-2] team. +

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job

@@ -827,17 +827,17 @@ Priorities: High (must have) - `* * \*`, Medium (nice to have) - `* \*`, Low (un
[width="59%",cols="22%,<23%,<25%,<30%",options="header",]
|=======================================================================
|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
|`* * *` |Software engineering project manager |Be able to add/delete projects in the app |Keep track of any new projects and remove unnecessary entries

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good ordering by priorities. Any one star user story you can think of?

@@ -879,7 +879,7 @@ _{More to be added}_
== Non Functional Requirements

. Should work on any <<mainstream-os,mainstream OS>> as long as it has Java `9` or higher installed.
. Should be able to hold up to 1000 persons without a noticeable sluggishness in performance for typical usage.
. Should be able to hold up to 1000 employees without a noticeable sluggishness in performance for typical usage.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try to quantify above average speed i.e. how many words per min

@@ -879,7 +879,7 @@ _{More to be added}_
== Non Functional Requirements

. Should work on any <<mainstream-os,mainstream OS>> as long as it has Java `9` or higher installed.
. Should be able to hold up to 1000 persons without a noticeable sluggishness in performance for typical usage.
. Should be able to hold up to 1000 employees without a noticeable sluggishness in performance for typical usage.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try add more NFRs

* *History* : `history`
* *Undo* : `undo`
* *Redo* : `redo`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please change this for your own product

cchj1995 pushed a commit to cchj1995/addressbook-level4 that referenced this pull request Mar 11, 2019
DMGREENHOUSE pushed a commit to DMGREENHOUSE/addressbook-level4 that referenced this pull request Mar 19, 2019
ditan96 and others added 30 commits April 15, 2019 21:34
added logging for commands and user story
Add warnig for reserved words
changed logging level for adding user story to FINE
Update sampleDataUtil.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants