icewizard4902 / ped Goto Github PK

Problem in the title.

Should this behavior not be allowed to avoid users executing wrong commands on the contact list accidentally?

The product is currently in the Assignment toggled view.

In this case, if we input lsmod, should the behavior be that the view is switched to contacts viewing instead of the assignment? Going through the process of typing view contacts then lsmod can be a bit of a hassle to users.

Command result:

I created a module named CS2106 with a class named T03.
These are the commands used.
mkmod CS2106
mkclass m/CS2106 c/T03

If the user wants to add a user in this module but of a class that does not exist (say T04) by this command add n/Vinh p/98765432 e/[email protected] m/CS2106 c/T04, should this displays a message saying that there does not exist any class in the CS2106 module with the given class name?

However the error message is saying that the module does not exist (which is not the case since I have already created a module named CS2106)

This might cause confusion to users using the product.

If we are in the CS2103T module and we try to search for Alex Yeoh,

this command should display this entry after the user types in find Yeoh.

We're in the CS2103T module.

This is the result after the command.

I suppose the find command behavior here should be that it searches for entries in the displayed list (which in this case is the list for students enrolled in CS2103T).

Hence, I think this classifies as a DocumentationBug rather than a FunctionalityBug as the UG does not specify what is the scope of the find command

An input of cd * (with two spaces after the cd) will result in the display of all the contacts.

Should the program restrict the input to be cd * (with one space after the cd)?

The data used is the sample data loaded when the product is first launched.

The mv 2 m/CS2103T c/T07 should display an error saying that the person is already in the module.

However, the command is successful.

Then after that we execute the command rm 2 m/CS2103T c/T07. Again, the intended behavior (if the product does allow duplicate module and class names) should result in all module and class being cleared for the contact at index 2.

The result is not the expected behavior though.

Should there be a check to ensure that there's no duplicate entries in the class and modules of each entry in the contact list?

The data used is the sample data of the product.

Input dlmod CS2103T. Should the behavior be that every entry with the module code CS2103T got the module deleted and displayed on the contact list?

This is the screen after that input.

This can be very confusing to the user.

Afterwards if the user types the command cd CS2103T this results in an error saying that the module does not exist, despite the entry #2 showing that it is still in CS2103T

Also, if we are in the toggled assignment view then execute dlmod CS2103 should the assignment list be wiped out as well? As assignments cannot exist if the module does not exist.

After dlmod CS2103 the displayed assignment list stays the same.

Since there is an entry in the assignment list to show the status of each assignment, should there be another way for the user to change the status from NOT_SUBMITTED to SUBMITTED

Also, should the grade command automatically set the status to GRADED?

It is not too clear how the index functions here (zero-based, one-based, ...). This might confuse users as there is not any index of the assignments displayed on the Assignment view.

Some features do not have example commands given for illustration.

For example, user might find it a bit confusing in the Grade feature what to input in the STATUS portion.
An example command here is appreciated!

The data used is in the sample data.

Initial status of the product:

If we input mkasg n/lol w/100 s/50 then mkasg n/lol1 w/100 s/50, the latter command should not be allowed logically the sum of the weightage of all the assignments should not be greater than 100%

However this behavior is allowed.

Should there be a check to prohibit this?

This is more like a suggestion for the grade feature.

Typing the name of the assignment repeatedly can be a hassle to users using the product. In actual grading a user might not want to type the same assignment name for a class of hundreds of students.
This product has a really nice view of all the assignments the students have.

Maybe you guys can implement index for the assignments of each student in the module (to replace the n/) so that the user can avoid typing the same name repeatedly and improve efficiency?

The behavior of the clear command in the product is that it wipes out the contact and the assignment list, but the modules and classes are retained.

Should the documentation here be a bit more specific of the scope of the clear command? Clearing all entries can imply that modules and classes are wiped

The data used is the sample data of the product.

We are in the CS2103T module now.

According to the user guide, the behavior when I input the command rm 2 m/CS2103T c/T07 be that the second entry be removed successfully? 2 is the input of the "displayed" list.