crypto-code/pe

pe's Issues

Single Word command takes arguments

It is possible to execute commands like list, undo, redo don't show error if extra strings are provided.

Add command error message

If we run the following command,
add n/John Doe m/CS2100 t/Lab04

It prints out an invalid command format message.

Although here the format is correct, just that the required field is missing.

Inaccurate Sequence Diagram [Page 6]

Here when delete is called, there should also be a call to the model.commitAddressBook() method since delete is an undoable command.

Inconsistency in UG and DG

According to the User Guide,

But in the DG (and also while running the app), it says that the giveall command will allow this but will not duplicate assignments. The command will only show error if all students in that mod already have the assignment assigned to them

No Date Validation

I am able to give assignments with due dates that are before today's date.

Help Window not shown if Minimized

Once the help window is minimized, clicking on help or entering the help command will not show the help window.

Inaccurate Sequence Diagrams for Feature Implementations

In the feature implementations of undoable commands like give, giveall and find, the sequence diagrams don't contain the call to model.commitAddressBook() which helps facilitate the undo and redo commands.

Very Long Tags are cutoff