sugiyem / pe Goto Github PK

The app doesn't allow user to add schedule with more than 3 hours duration. However, there is actually some course in NUS where the lecture's duration is more than 3 hours (Take CS3233 for example)

The system allow invalid email, such as roy@com to be inserted.

There might be some student which doesn't have any telegram account. As a result, user can't add this student to the app as it is required to fill the telegram handle.

The diagram of Model component in the DG is unreadable as it is too small.

The system allow invalid phone number, such as shown below, to be inserted.

In the UG, specifically section 4.4.1, there is a typo. Tutoria instead of Tutorial

The sequence diagram of medit in the DG is unreadable as it is too small.

test

test

The sequence diagram of editstu and medit in the DG is unreadable as it is too small.

In the UG, there is an inconsistency in hyperlink. For example, in the picture below, the bluish Student Manager Command Parameters are clickable and it will bring user to the expected section. However, the bluish Editing a Student is not clickable.

I feel like some of the commands, such as add and sadd has too many required fields. As a result, it become quite hard to use and test.

In the UG, the term one-based is used. It'd be better if this term is elaborated more clearly as reader may not understand it.