arjavibahety / ped Goto Github PK

Unable to test the remind command because the next view time is set to after 24 hours. Maybe consider bringing it down to 30 seconds or something for testing purposes?

According to the user guide, the user will be prompted to either type y or n to confirm deletion. However, the app requires user to hit enter key instead. Currently, there is no explicit command to refuse deletion. Perhaps it would be better to state that just to make things clearer?

I think you meant timetrail [time/TIME] [tag/TAG]... instead?

When adding a new item in the cheatsheet, the tag used was cs2100midterm. Unable to add cs2100 tag / edit the existing tag

UG mentions that items indicated in the square brackets are optional in section 3. However, in section 5.7, tag is enclosed in the bracket and the user is told that at least one tag must be specified.

In fc, multiple entries for the same field is not handled. For example, the user should not be allowed to give multiple answers for the same qn. Maybe you can consider either throwing an expection or concatinating multiple resposes for the same field into a single string?

Tags are not shown.

It would be better if the user guide specified that tags are case insensitive. For instance, in this case, HelloWorld and helloworld are considered as the same tag.

Documentation needs to specify that the user needs to use switch for switching tabs.

Missing numbering in expected output in the user guide

I understand that in the beginning of the UG you specified that all the commands need to be alphanumeric. However, add command in fc is allowing punctuations whereas notes does not. Perhaps add more details in the UG or standardise the features to allow punctuations?

Additionally, if you do not want notes command to handle punctuations, it would be better to throw an exception to prompt the user instead of not displaying anything?

I realised CheatSheet is indeed throwing exception. Maybe follow that?

add command in fc allows duplicates but does not allow in notes. Perhaps you want to standardise it?

Otherwise, maybe clearly state in the UG that notes does not allow duplicates and fc does?