add /g Poker /s 500 /e 650 /d 20/06/21 /dur 142 /loc Sentosa Casino command provided in the UG does not work

Labels: severity.Medium type.DocumentationBug
original: ryantianj/ped#2

Right now our class name is GameType, but we use GameName for our user guide and messages to user.
To make it more consistent, we should change it in v1.4.

Places to look for change:

• GameEntry toString() method
• GameEntry unit tests
• Strings in Command files
• UG

1. On the first start, most of the graph and all of the statistics cannot be seen. I suggest changing the default window size or enforcing a minimum window size. e.g.:

2. On smaller screens, statistics cannot be seen. For example, this is how the GUI looks on my laptop screen (1280x720):

Labels: type.FeatureFlaw severity.High
original: Chesterwongz/ped#1

Allow users to enter help commands that are specific to different features.
Eg. 'help add' should show what is the format of an add command, and explain what an add command does.

The general 'help' command without any argument should be updated to show a command summary of all the commands available in GameBook.

... so that I can filter my games based on location, date and time

... so that I can enter information which I forgot to previously enter

Up till now, GameBook clears user's data once it receives the command "clear."
This is not user friendly and is unsafe, as GameBook does not have implementations of backing up data.
A confirmation pop up window will make the execution of ClearCommand more user friendly.

Unable to add a game entry

Steps:

• Start application
• Insert the following add command taken from the User Guide Provided:

Command: add /g poker /s 0.01 /e 1.02 /d 11/9/21 21:20 /dur 3:14 /loc Home

Incorrect Error Log: Cash values should be floating point numbers with at most 2 decimal places.
Should be Command Success I think.

Labels: severity.High type.FunctionalityBug
original: Siddharth-Sid/ped#1

Parsers should then catch the unchecked exception IllegalArgumentException if arguments fail.
Certain possible failure include missing gametype, missing end amount, having both start end and profit amounts.

Recent parser-subclass changes broke it. I think just the Tag part is broken. I'll take a look.

... so that I can view game details from previous games upon opening the app

(while calling List)

Each member updates their individual AboutUs pages

Within integrate-changes-branch.

Currently we made some changes to command arguments.
Let's reflect that in the prompts.

e.g.
add: Adds a game to the game book. Parameters: /g GAMENAME /s INITIALCASH /e FINALCASH /date DATE /dur DURATION /loc LOCATION [/tag TAGS ... ] Example: add /g blackjack /s 200 /e 250 /date 03/10/21 /dur 50 /loc 311, Clementi Ave 2, #02-25 /tag friends

Each person to update what they previously worked on

There seem to be some issues with markdown, I think this was supposed to be in a table format but it is not showing on the page.

Labels: severity.Low type.DocumentationBug
original: peilinye/ped#1

... so that I can include any game I want instead of choosing from a list of common games (Note: to add as a tag for now)

Reset LineChart series upon calling the Clear command

The app launches with weird screen dimensions which cuts off some data

Labels: severity.Medium type.FeatureFlaw
original: ryantianj/ped#1

Have only String as API argument in Model subclasses, and move checking for validity into Model subclasses. Create subclass wrappers for all model components.

... so that I can access the relevant details of my games

"Profit" is accepted as an argument for add command, as it is more user friendly and intuitive than the previous required inputs: start amount and end amount.
UI is changed to reflect profit per game entry, instead of showing the start amount and end amount.

Issue to consider:
Should we still keep start amount and end amount in game entry objects?

Change existing parser to parse Game Entries.

duration: Change from /duration to /dur and add new allowable formats
location: Change from /location to /loc
game: Change from /game to /g
start: Change from /start to /s
end: Change from /end to /e

The current format requires us to have multiple "/tag" to add multiple tags, for example, "... /tag tag1 /tag tag2 /tag tag3". Change to allow "/tag tag1, tag2, tag3".

Implement basic GameEntry and GameBook classes

... so that I can clear entries that have erroneous data