s7u4rt99 / pe Goto Github PK

Screen Recording 2021-11-12 at 4.39.29 PM.mov

As you can see from the video, the link at the words Contacts Panel is not working.

Following the order of the getting started, after opening the app, I typed contacts, then tasks then add n/ruwheuh3irh but it throws this error. Maybe it can be clearer if we are only allowed to use the add n/NAME command in the contacts tab or tasks tab.

The diagram above is very messy, there are so many arrows going into UiPart. Perhaps you can take out the items in the TabMenu and put it in another diagram to make it more readable and understandable.

UG says If i specify multiple Date Time, only the last would be taken. However, in this case, it is not accepted when i do add d/test dt/12/02/2021 dt/19:00 PM

Apologies, this is not a bug. However maybe you can say it in the error message to remind users that it is a 12 hr timing when we use AM or PM

Maybe you can split it into smaller Diagrams

Screen Recording 2021-11-12 at 4.16.53 PM.mov

I can click anywhere on the sentence for the link, I should only be able to click it on the words Not sure what java is? Click here

I am not sure why my input is not accepted when i do add d/test dt/19:00 PM, but is accepted when i do add d/test dt/12:00 PM. This makes it unusable if only certain times are allowed.

EDIT: Apologies, because it is in 12 hour clock. Maybe you can include it in the error message to say that the input is not in 12hr format.

Assignees still displayed even though i ran clear done

As you can see, the first picture brings me to the second picture, and under step 3 says to run seedu.address.Main. However, there is no seedu.address.Main in your github repo for me to run. It should be tp/src/main/java/dash/Main.java. This is a very serious issue as users are not able to set up properly now.

Based on google (i googled it during PED), alphanumeric numbers consists of some basic symbols (* & @) if i remember correctly. However, this does not accept them.

Under point 3, it should be by reading about Dash's architecture. instead of by reading about AddressBook’s architecture.. This will make the user confused

Missing [else] in activity diagram. Should include for clarity.

Allows to add alex yeoh even though Alex Yeoh is inside. If i remember the cs2103t website says that they should be considered the same person even though the letters are not in capital letters.

test test 123

By clicking the link above (first pic), it brings me to the webpage above (second pic), which does not explicitly state where to download java11.

As a mac user, i clicked the link above (third pic) which brought me to the page below, and it only has java 8 not java 11.

Without the correct link, the application may not work properly on their devices and cause issues. Thus I flagged this as High.

Screen Recording 2021-11-12 at 5.04.39 PM.mov

When i click the classes, the assignees on the right does not get updated and display the correct assignees