domoberzin / pe Goto Github PK

When specifying a date with the time set as 2400, the UG specifies the behaviour highlighted below.

However, when trying to execute a command with that timestamp, this error is observed.

In this particular class diagram, each arrow is dashed, while in the notes it is mentioned that dashed lines are use for interfaces/abstract classes. There is also a dashed arrow where XYZ command implements an abstract command class. This may confuse the reader as he or she may infer that all of these classes are abstract or are interfaces, despite the different arrowheads.

When adding a task, a deadline from the past can be specified, without any warning or error. However, there is also no documentation in the UG that such behaviour is not blocked.
As seen below in task 1, the deadline is from the year 2000.

Overall, there is no clear indication that duplicate entries for contacts and tasks are not allowed in the commands in the UG, but these commands will be rejected by the application.

When executing a add task command whose name contains a comma and apostrophe, the command is blocked. For e.g. add_task n/,' d/\\\$Foo would result in an error. However, in the DG, this behaviour is mentioned and is apparantly allowed. Setting this as high since it was a behaviour that was specified to be allowed, but resulted in an error.

When switching between task and name pane, the autocomplete suggest remains the same. Not major but could be more user friendly by changing to add_task and add_contact where necessary.

As in the title, there are very little images of sample command executions in the UG to show the user how commands are executed and their results.

In the two diagrams below, deletePerson is called differently. One of them calls it utilising the specified index (deletePerson(1)), while the other calls it using a variable named p (deletePerson(p)). This inconsistency could confuse the reader.

Tags containing multiple words are blocked, and the error message given does not give a good indication to the user that the tag needs to be a single word. Additionally, I am unable to find any documentation in the user guide that multiple words in tags are not allowed

As a project lead, one may have many different tasks (i.e. 100 tasks for example). There is currently no good way to hide the completed tasks other than to delete them upon completion. If there are many completed tasks, this can clutter the interface of the user and make it tedious to navigate through to find incomplete tasks, hindering the user's ability to track tasks. Additionally, the user may want to refer to a completed task in the future, and deleting it to reduce clutter would prevent the user from doing so.

NFR of the DG also states that the app can support up to 1000 tasks. In such an event, I don't think it's feasible for the user to navigate through the list of tasks by simply using the find feature.

When inputting multiple name tags, the application will simply add in the second name specified.
For example. the command add_contact n/John n/Mike p/ 123 e/ [email protected] a/ None would result in a contact named Mike being added.
There is no documentation of this in the user guide, and no error message to indicate that 2 names have been inputted, but this command is not blocked.
This may result in the user unintentionally assigning the wrong name to a contact.

There is only one sequence diagram of a sample execution of the delete contact command. There is no such diagram for other commands such as adding a contact, adding or deleting a task etc. May be good to have some for the different commands so the reader can see the differences between the various commands' execution

When adding a contact, it is necessary to specify the address of an individual. However, the application is for project management, so I'm not sure how having a mandatory address field aids in project management (i.e. assigning tasks etc). It makes entering a new user more arduous as more tags need to be specified.

When adding a date that does not exist, the error message does not indicate appropriately that the date is invalid.
For example, in the image below I used 29-02-2022 2359, which does not exist, but the error message is telling me the format is invalid. This could hinder the user in the event that he or she inputs a date that does not exist. Additionally, nothing was mentioned in the user guide with respect to dates.

As stated in title, when I delete a contact with some tasks that are assigned to them, the tasks get deleted as well. However I am unable to find documentation of this behaviour in the UG.

When adding a person with a duplicate name, phone, and email, the command is blocked.

However, when adding a person with a different name, but a duplicate phone number or email, the command is not blocked. This is a little bit confusing as there's no documentation on duplicates in the UG, but having duplicate phone numbers and emails makes no sense, as emails and phone numbers are meant to be unique. Example shown below, as you can see, there's duplicate phone numbers and emails across differently named individuals.

Hi this is a dummy bug
Hi this is a dummy bug
Hi this is a dummy bug
Hi this is a dummy bug
Hi this is a dummy bug