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

Update Developer Guide User Stories, Use Cases section #13

Merged

Conversation

zhaojj2209
Copy link

Partially addresses #9.

Added target user profile, value proposition, user stories and use cases.

@zhaojj2209 zhaojj2209 added type.documentation 🧻 Improvements or additions to documentation priority.medium 🥈 Todo for current iteration pr.ready-for-review 📝 labels Sep 21, 2020
@zhaojj2209 zhaojj2209 added this to the v1.1 milestone Sep 21, 2020
@zhaojj2209 zhaojj2209 requested a review from a team September 21, 2020 15:37
docs/DeveloperGuide.md Outdated Show resolved Hide resolved
docs/DeveloperGuide.md Outdated Show resolved Hide resolved
docs/DeveloperGuide.md Outdated Show resolved Hide resolved
@@ -292,15 +297,14 @@ Priorities: High (must have) - `* * *`, Medium (nice to have) - `* *`, Low (unli
### Non-Functional Requirements

1. Should work on any _mainstream OS_ as long as it has Java `11` or above installed.
2. Should be able to hold up to 1000 persons without a noticeable sluggishness in performance for typical usage.
2. Should be able to hold up to 1000 expenses/income 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.

expenses/incomes instead of expenses/income


**Value proposition**: manage finances and cultivate good financial habits (such as saving) efficiently by typing in CLI commands

Choose a reason for hiding this comment

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

Manage instead of manage

@ianyong
Copy link
Member

ianyong commented Sep 21, 2020

LGTM!

Copy link

@yongping827 yongping827 left a comment

Choose a reason for hiding this comment

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

LGTM

@yongping827 yongping827 merged commit 1e70ae1 into AY2021S1-CS2103T-W16-3:master Sep 21, 2020
@zhaojj2209 zhaojj2209 deleted the jingjing/developer-guide branch October 16, 2020 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.medium 🥈 Todo for current iteration type.documentation 🧻 Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants