yanchenglee98 / ped Goto Github PK

Minor nit, but edit feature allows for commands which have no change.

i.e.

1. add n/tP PE m/CS2103T des/Practical Exam
2. edit 1 n/tP PE

Would be good to notify user that no change has been made to the task instead.

In the UG it says that:

but in the app 2101 will match CS2101

Would be good to confirm which is the intended behaviour

Minor nit but some details such as name and progress percentage of some task is truncated in the task list when in the default windown size.
Would be good to the event cards scalable i.e event card size increases as window size decrease
Or move details to a new line and have the text wrap around.

UG does not have a glossary to explain technical terms

Despite having CS1101S in the module list, CS1101s was also addedd to the module list. Since most module codes in NUS are capitalized, module code can be set to be case insensitive.

Would be good to add a note explaining that Assigning a task to a module that already has a module assigned to it will just replace the old module with the new module

Deletion of a task from the task list does not delete the linked task in countdown

For example if the task "tP PE" is deleted after delete 3 has been executed

The task in countdown is still there

If there is an event with a deadline of today and time of 2359 such as event 1 Meeting

Adding it to countdown will result in an error saying that it is in the past i.e. countdown task/1

Would be good to compare the deadline timing also. Or specify in the UG that this scenario is considered as past