• parses quiz commands
• edit LogicManager to switch between flashcard mode and quiz mode (to ensure that flashcard commands cannot be accessed in quiz mode and vice versa)

1. The first line says "DSAce is a desktop app for managing contacts, optimized for use via a Command Line Interface" .

Should it be changed to "a desktop app for managing flashcards for revision"?

2. Under the instructions for the edit command

Should "person(s)" be changed to "flashcard(s)"?

Labels: severity.VeryLow type.DocumentationBug
original: tigerting98/ped#1

No details provided.

there is no function of ability to attempt the quiz. the UG does not describe what the quiz mode feature does or how to utilise it. Seems like the feature is incomplete and should not have been merged into their main repo.
Can consider publishing a more complete description of the quiz function in the UG and implement some functionality for the quiz function as right now it does not accomplish much

Also, given that the UG states that this is a contact managing app, i am confused as to what a quiz mode would accomplish. Could specify in the UG that maybe the quiz mode is to test how well the user knows his contacts

Labels: severity.High type.FunctionalityBug
original: Christopher-LM/ped#14

• [Task] Implement delete and amend delete command, delete parser in the code

No details provided.

perhaps the edit command should return an error message if trying to edit the same parameter more than once. in the screenshot above, the edit command returned the second name parameter of 'test12'

Labels: severity.Medium type.FeatureFlaw
original: Christopher-LM/ped#8

No details provided.

It would be good for the add section under the UG to include that the priority of a new contact added would be defaulted to low if left blank unless this was not intended.

Labels: severity.Low type.DocumentationBug
original: Christopher-LM/ped#4

data sample needed to demo v1.2

• Amend add command class and add command parser

No details provided.

the introduction states that the app is designed for use of saving a contact but the entries in the address book does not save important information such as mobile number or email of that sort. perhaps can consider saving other fields as well and updating the UG accordingly.

Labels: severity.High type.DocumentationBug
original: Christopher-LM/ped#13

I realized that you guys choose to make priority optional in the add command and give it a default value of low if not specified. I think it's fine to do that provided that you state it in your UG so that users don't get confused.

Labels: severity.Low type.DocumentationBug
original: Stefan-Jiang/ped#4

No details provided.

clear command should return an error message when the input contains more than "clear" for example "clear 444" works as well.

Labels: severity.High type.FunctionalityBug
original: Christopher-LM/ped#11

According to UG, the tile of UI should be DSAce, the actual is "Address App".
The jar file name should be dsace.jar, the actual is "addressbook.jar".

Labels: severity.Low type.FunctionalityBug
original: Stefan-Jiang/ped#1

No details provided.

UI could be improved to wrap the text when the description of a flipped card exceeds the width of the UI box.

Labels: severity.Low type.FeatureFlaw
original: Christopher-LM/ped#6

No details provided.

The UG states that the find feature does not return result when the keyword exists as part of the parameter but it would be good if it could find partial keywords as well

Labels: severity.Low type.FeatureFlaw
original: Christopher-LM/ped#1

• Renaming it to be 'UniqueFlashcardList'
• Renaming its methods and references to UniquePersonList class as appropriate
• Renaming DuplicatePersonException and PersonNotFoundException classes referenced in UniquePersonList

Replace instances of the Address class with the Definition class where necessary

Implements enter/exit quiz parser to support change from flashcard to quiz mode.

No details provided.

GUI displays the flipped flashcard information when user flips the card back down.
Would be good to have a new message displayed when the card is flipped back down instead of displaying the card information when flipping up.

Labels: severity.Low type.FeatureFlaw
original: Christopher-LM/ped#2

UI seems to be lacking a bit, not much change from the original AB3 UI, would be good to improve it and perhaps make it more aesthetic, other than that it looks gtg!

Labels: severity.VeryLow type.DocumentationBug
original: Christopher-LM/ped#10

Create following classes:

• Response: contains question and user's answer, checks if user answer is correct
• Attempt: records all responses, calculate score
• Performance: records all attempts, calculate statistics of all attempts

As a user who has just started to use the app, I can view the user manual to get a brief idea of how to use the app.

As a user ready to start using the app, I can purge all current data, so that I can get rid of sample/experimental data I used for exploring the app.

hehe done

In UG, the data should be stored in a file called DSAce, the actual product stores data in data/addressbook.json instead.

Labels: severity.Low type.FunctionalityBug
original: Stefan-Jiang/ped#6

In UG, the jar file name should be dsace.jar, the actual is “addressbook.jar”.

Labels: severity.Low type.FunctionalityBug
original: Stefan-Jiang/ped#7

I am aware that you guys disallow all commands except leave quiz in the quiz mode, can probably consider allowing exit/help command in the quiz mode to increase user-friendliness.

Labels: severity.Low type.FunctionalityBug
original: Stefan-Jiang/ped#3

I noticed that you guys allowed sorting without specifying the order and give it a default value of ASC. I think it's okay to do that but you probably need to tell users about it.

Labels: severity.Low type.DocumentationBug
original: Stefan-Jiang/ped#5

The extension should be .png (not .PNG)

To consider a milestone properly wrapped up, it needs to be closed.

As a familiar user, I want to search for flashcards containing certain keywords.

• Main window should change appearance when quiz functionality is activated
• QuizCard for GUI display
• Toggling GUI between flashcard and quiz modes

As a new user, I can make a few flashcards by creating each flashcard one-by-one to experiment with the app.

As a second-time user, I can make new flashcards about the definition of a concept so that I can remember it better.

Help Guide

• Changing link URL to our own help guide

Name Class

• Renaming it to be 'Title'
• Renaming all its references to be 'Title'
• Slightly changing the code to be more suited to be title

This phrase describes AB3, not applicable to your project.

When I added a new entry in the list and tried to use the flip command, it gave me an unknown command error message which should not be the case (should probably at least say something like "this one cannot be flipped")

Labels: severity.Medium type.FeatureFlaw
original: Stefan-Jiang/ped#2

No details provided.

After flipping card at index 3, followed by exiting and reopening the jar, the card is not flipped and the user guide seems to indicate that commands which changes data would be saved, perhaps it would be good to save the state of the card as well.

Labels: severity.Low type.FeatureFlaw
original: Christopher-LM/ped#3

No details provided.

name should not take in other inputs like numbers. does not make much sense, should return an error message

Labels: severity.Medium type.FeatureFlaw
original: Christopher-LM/ped#12

Flip command for flipping the flashcard/definition.

No details provided.

user input of 'find z' displays the entire contact book even without specifying the parameter for the find function. Furthermore, there is no letter 'z' in any of the entries displayed. Would be good to display an error message when the parameter is specified and not display any results

Labels: severity.High type.FunctionalityBug
original: Christopher-LM/ped#5

1. Able to execute command without using the relevant tokens such as n/ or t/ as well as multiple copies of the same token

From the screenshots above, I was able to enter the command without any tokens, and the application simply displays everything as it was an invalid search term

I could also use multiples of the same token, and be given an empty list. This might be because the programme treats the repeat token+search term as 1 keyword to search, but that would not be consistent with the second bug shown below.

2. Unable to execute command when there are multiple keywords in the n/ token.

I was unable to search for any flash cards including "sort" or "quicksort". Pressing enter simply does not do anything.

From the user guide, it seems that I should be able to enter multiple keywords

Labels: severity.High type.FunctionalityBug
original: tigerting98/ped#2

No details provided.

The displayed error message of "Example: edit 1" is incomplete and will result in an error if the user inputs that command, instead some parameters could also be specified in the edit command as well maybe "edit 1 n/test1" would be better

Labels: severity.Low type.FeatureFlaw
original: Christopher-LM/ped#9

As a user with some limited free time before the bus arrives, I wish to make good use of my time. I want to refresh my memory of CS2040s contents on flashcards while waiting for my bus.

As a new user who is prone to making mistakes, I can simply delete flashcards containing errors and then create new ones again.

As a second-time user, I can view the user manual again for a reminder of how to use the app.

No details provided.

the example shown when an add command is given is incorrect and should be updated accordingly

Labels: severity.Low type.FeatureFlaw
original: Christopher-LM/ped#7

• Rename Person class to Flashcard class
• Rename all references to Person class as appropriate
• Remove unnecessary fields in constructor of Flashcard object