-
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
Add Reset Scores feature, make GUI bigger, and update UG #236
Add Reset Scores feature, make GUI bigger, and update UG #236
Conversation
@@ -35,6 +35,7 @@ public CommandResult execute(Model model) { | |||
model.setRandomQuizMode(false); | |||
} | |||
model.saveScore(); | |||
System.out.println(model.getQuizTotalQuestions()); |
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.
I'm guessing you added this for debugging? Should probably get rid of it if so
@@ -122,6 +123,13 @@ public Command parseCommand(String userInput) throws ParseException, ParseZeroEx | |||
} | |||
throw new ParseException(MESSAGE_UNKNOWN_COMMAND); | |||
|
|||
case ResetScoreCommand.COMMAND_WORD: | |||
System.out.println(arguments); |
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.
I'm guessing you added this for debugging? Should probably get rid of it if so
…into Add-Storage-File
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.
LGTM
Fixes #193, #175, #200, and #208.