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

[CS2113T-F11-2] BudgetTracker #45

Open
wants to merge 639 commits into
base: master
Choose a base branch
from

Conversation

WeiXuanYap
Copy link

Target User: Tech-savvy students who have trouble keeping track of their expenses.
Value Proposition: Set guidelines for our users' spending habits so they do not exceed their budget and allow them to cut down on spending on areas which they tend to overspend.

Copy link

@APZH APZH left a comment

Choose a reason for hiding this comment

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

Week 11 Tutorial Review


### Value proposition

{Describe the value proposition: what problem does it solve?}
Value proposition: allow users to record and track expenses more conveniently,
and thereby encourage them to have greater control over their finances.

## User Stories
Copy link

Choose a reason for hiding this comment

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

Should the user stories be sorted by version or type of user? Would it be neater and clearer?


{Describe the design and implementation of the product. Use UML diagrams and short code snippets where applicable.}

## Implementation
###Delete feature
Copy link

Choose a reason for hiding this comment

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

Missing a space after the '###', resulting in 'Delete feature' to not be identified as a header.

<br/> **Step 5**. The Parser starts to parse the command, it extracts starting index 3, ending index 5, and month 10. As this is a range of expenditures to be deleted, the Parser class calls DeleteMultipleExpenditureCommand to work. By using a for loop, the 3 expenditures are successfully deleted and prints out a showMultipleExpenditureDeletedMessage from TextUi class.


###Edit feature
Copy link

Choose a reason for hiding this comment

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

Missing a space after the '###', resulting in 'Edit feature' to not be identified as a header.


## Design & implementation
## Design
Copy link

Choose a reason for hiding this comment

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

May I suggest the inclusion of UML diagrams to supplement the explanation of the features stated below, on top on the step by step guide currently included.

LS-Young and others added 30 commits November 12, 2021 12:31
Add Introduction, Product Overview and Budgie in edit section
Update emoji in list in UG
Add identified tag table + Edit TOC
…into UserGuide

# Conflicts:
#	docs/UserGuide.md
Add emoji to delete part and page breaks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants