shirsho-12 / pe Goto Github PK

Edit Command has an incorrect Error Message for the user. Should be the same as that for -1.

Time slots for meetings can often be until midnight or even later. Our meetings were usually from 10 pm to 1 am. This cannot be inserted into coNtactUS while a meeting from 10 pm to 11:59 pm is ok. This is especially a problem for the user as many computing students have meetings at these times.

Error:

But this is OK:

The Work Today prompt, although a very interesting and useful prompt, has a minor bug. The prompt can be made full-screen even though it doesn't need it. The full-screen mode doesn't look as nice.

Disabling the full-screen box would, in my opinion, make it better.

The add command cannot detect duplicates in spacing in the name field.

Steps to recreate:

1. Run add n/CS2103T t/Lecture
   Output:

2. Run add n/CS2 103T t/Lecture
   Output:

3. Run add n/C S 2 1 0 3 T t/Lecture
   Output:

Result:

To my understanding duplicates are not allowed according to the following error.

The find command currently goes through all the parameters and matches according to whatever substring matches. However, this functionality may be suboptimal if the user just wants to find on a specific field.

Example:

1. Add data.

2. Run find t

Finds everything

This is problematic if the user wants to label his/her tutorials as T and lectures as L. Searches are, in this case, unusable.

Example:

The add command cannot differentiate between spacing in tags.
Steps to recreate:

1. Run add n/CS2103T t/Lecture
   Output:

2. Run add n/CS2103T t/Lec ture
   Output:

3. Run add n/CS2103T t/L e c t u r e

The add command's address field allows invalid input to be put in.

Steps to recreate:

1. Input add n/CS1010E t/Tutorial e/Monday 10:00 12:00 a/@#!@$# s/Sam Wan

The statement

The software size (including the data file with up to 1000 modules) should not exit 100MB.

Should be

The software size (including the data file with up to 1000 modules) should not exceed 100MB.

Steps to recreate:

1. Normal command: `add n/CS2103T t/Lecture

2. Erroneous command: add n/cs2103t t/Lecture

Both should be treated as the same but are considered different:

List command should be able to get the previous ordering back, or the very least there should be a command to set a custom ordering of the modules.

Steps to recreate:

1. Run sort timeslot
   Output:

2. Run sort deadline
   Output:

3. Run list
   Output:

As stated in Issue-9, sorting by timeslot and deadline changes the order of inputs.

This is in contrast to what was stated in the DG:

Example:

1. Run sort timeslot

Output:

2. Run sort deadline

Output:

The organization name for the code should have been changed from SE-EDU to something else. This can be seen in the following line:

Running sort by timeslot or sort by deadline changes the order of modules despite no times set.

Steps to recreate

1. Run sort timeslot
   Output:
   

2. Run sort deadline
   Output:
   

This is confusing to the user as there is no indication as to why the orders changed on running the commands.

The UG states that t/ is for the description but the UI shows that t/ is for Tags. This is confusing to the user. I was initially confused after reading the UG for this part as well.

UG:

Application: