-
Notifications
You must be signed in to change notification settings - Fork 4
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 deck #23
Add deck #23
Conversation
import seedu.address.logic.commands.RedoCommand; | ||
import seedu.address.logic.commands.SelectCommand; | ||
import seedu.address.logic.commands.UndoCommand; | ||
import seedu.address.logic.commands.*; |
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.
Please undo this and configure intellij such that it does not automatically collapse import statements.
@@ -10,4 +10,10 @@ | |||
public static final String MESSAGE_INVALID_CARD_DISPLAYED_INDEX = "The card index provided is invalid"; | |||
public static final String MESSAGE_CARDS_LISTED_OVERVIEW = "%1$d cards listed!"; | |||
|
|||
public static final String MESSAGE_DUPLICATE_DECK = "This deck already exists in TopDeck!"; |
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.
Change to "A deck with this name already exists!"
* Returns an unmodifiable view of the filtered list of decks | ||
*/ | ||
ObservableList<Deck> getFilteredDeckList(); | ||
|
||
/** Returns an unmodifiable view of the filtered list of persons */ |
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.
Help to change persons
to cards
public ObservableList<Card> getFilteredCardList() { | ||
return model.getFilteredCardList(); | ||
} | ||
public ObservableList<Deck> getFilteredDeckList() { return model.getFilteredDeckList(); } |
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.
Don't put everything in a single line. Intellij shows code in a single line to save vertical space but the underlying code is actually formatted properly.
attempt for deck feature