forked from nus-cs2113-AY2324S1/tp
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Javadoc ndg #197
Merged
wwweert123
merged 7 commits into
AY2324S1-CS2113-T18-2:master
from
wwweert123:javadocNDG
Nov 9, 2023
Merged
Javadoc ndg #197
Changes from all commits
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
2f2a764
Add javadoc to classes and methods
wwweert123 8660d42
Add DG value proposition and user profile
wwweert123 e3d3d00
Add user stories
wwweert123 1cbae2b
Add term to glossary
wwweert123 e901e67
Add checkstyle fixes
wwweert123 dc31f33
Add simple manual testing and fix bugs
wwweert123 cae5827
Update contributions for PPP
wwweert123 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -338,30 +338,90 @@ Example: `budget view` | |
## Product scope | ||
### Target user profile | ||
|
||
{Describe the target user profile} | ||
Our target user profile is ... | ||
- a working adult with a source of income | ||
- someone who dislike navigating graphic user interface | ||
- someone who can type fast | ||
- someone who cannot manage their finances such as income and expenses properly | ||
- unable to reach their financial goals | ||
- is slightly interested in the equity market | ||
- needs reminders for tasks | ||
|
||
|
||
### Value proposition | ||
|
||
{Describe the value proposition: what problem does it solve?} | ||
Our financial planner application can help individuals manage their finances effectively and achieve their financial | ||
goals. The purpose of such an application is to provide users with a range of tools and features to help them better | ||
understand their financial situation. This will enable them to make more informed decisions, and plan for their future | ||
financial well-being. The application will allow the user to keep track of their income, expenses and overall balance. | ||
It also lets the user view their income and expenses using visualization tool to have a better view of their cash flow | ||
based on categories. It also allows the user to set the budget for the month. It also allows users to add their financial | ||
goals to the wishlist. Furthermore, it allows users to track the stock market if they have interest in investing in | ||
equities. | ||
|
||
## User Stories | ||
|
||
|Version| As a ... | I want to ... | So that I can ...| | ||
|--------|----------|---------------|------------------| | ||
|v1.0|new user|see usage instructions|refer to them when I forget how to use the application| | ||
|v2.0|user|find a to-do item by name|locate a to-do without having to go through the entire list| | ||
| Version | As a ... | I want to ... | So that I can ... | | ||
|---------|-----------------------|-------------------------------|--------------------------------------------------------------------------------------| | ||
| v1.0 | user | Add my income | Store my income information and view/track them later | | ||
| v1.0 | user | Delete my income | Remove the income entry that I have mistakenly added or do not keep track | | ||
| v1.0 | user | Add my expense | Store my expense information and view/track them later | | ||
| v1.0 | user | Delete my expense | Remove the expense entry that I have mistakenly added or do not keep track | | ||
| v2.0 | user | set my expense type | Break down my expenses into different categories | | ||
| v1.0 | user | set my income type | Break down my income into different categories | | ||
| v2.0 | user | Add recurring cash flows | add a regular expense or income (salary, rent) easily | | ||
| v2.0 | user | Delete recurring cash flows | delete a regular expense of income easily | | ||
| v1.0 | user | list all cash flow entries | view all my income and expenses in a comprehensive list | | ||
| v1.0 | user | list all expenses entries | view all my expenses in a comprehensive list | | ||
| v1.0 | user | list all income entries | view all my income in a comprehensive list | | ||
| v2.0 | user | list all recurring cash flows | view all my recurring income or expenses in a comprehensive list | | ||
| v2.0 | new user | see usage instructions | refer to them when I forget how to use the application | | ||
| v1.0 | user | set a budget | keep track of a budget together with my cash flow and ensure I do not exceed it | | ||
| v1.0 | user | update the budget | make changes to the budget according to my needs | | ||
| v1.0 | user | reset the budget | return to my initial budget easily | | ||
| v1.0 | user | delete budget | remove the budget that I no longer want to keep track of | | ||
| v1.0 | user | view budget | keep track of the amount of budget I have left | | ||
| v1.0 | user | see overview of the app | see the overall view of all income, expense and overall balance as well as reminders | | ||
| v1.0 | user | view balance | see my overall balance according to the income and expenses I am keeping track | | ||
| v1.0 | investment enthusiast | view my watchlist | keep track of stocks that I am interested in | | ||
| v2.0 | investment enthusiast | add new stocks to watchlist | add new stock that I am interested in investing in | | ||
| v2.0 | investment enthusiast | delete stocks from watchlist | remove stocks that I am no longer interested in | | ||
| v1.0 | user | add reminder | add reminders (eg to pay loans) so I will not forget | | ||
| v1.0 | user | delete reminder | delete reminders that I no longer want to keep track | | ||
| v1.0 | user | mark reminder | set the reminder as completed | | ||
| v1.0 | user | view wishlist | keep track of my goals easily | | ||
| v1.0 | user | set goals | add a new goal to my that I think of | | ||
| v1.0 | user | delete goals | remove goals that I can no longer achieve | | ||
| v1.0 | user | mark goal | that I have achieved | | ||
| v1.0 | user | visualize my cash flow | easily see where the distribution for my spending and earnings | | ||
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. nice hard work |
||
|
||
## Non-Functional Requirements | ||
|
||
* Should work on main OS (Windows, Linux, Mac) that has Java 11 installed. | ||
* This app is meant for a single user. | ||
* This app is targeted towards users with an above-average typing speed. | ||
* Watchlist should work reliably and not crash the application when the 3rd party dependencies are down (API is down) | ||
|
||
## Glossary | ||
|
||
* *Cashflow* - Refers to an income or expense. | ||
* *WishList* - A list containing goals/targets. | ||
* *Watchlist* - A list of stocks that the financial planner is currently tracking | ||
|
||
## Instructions for manual testing | ||
|
||
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. Want to add this too? |
||
{Give instructions on how to do a manual product testing e.g., how to load sample data to be used for testing} | ||
Given below are instructions to test the app manually | ||
|
||
- Note: These instructions only provide a starting point for testers to work on | ||
|
||
### Launch and shutdown | ||
|
||
1. Initial Launch | ||
1. Download the jar file and copy into an empty folder | ||
2. Open up the terminal and run java -jar tp.jar Expected: | ||
shows you the welcome screen for the financial planner app | ||
2. Closing the application | ||
1. Type `exit` into the terminal. | ||
2. Expected: the financial planner will exit with a goodbye message. | ||
Under the data newly created data directory, a watchlist.json and a data.txt file will be created | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 details