jqchong / ped Goto Github PK

In my opinion, the show command seems rather extraneous since the extra information provided through the command is only the insurance plan and the profile picture. It does not help when each client can only have one insurance plan. If it does not affect the user interface by too much, maybe you can consider placing the insurance plan and profile picture under the client list instead?

person_icon.png is taken from vectorified. To be more exact, the icon was taken from the following website: https://vectorified.com/download-image#facebook-profile-icon-10.png .

It was stated clearly that attribution is required.

Right now it seems like the findmeet command will return extraneous meetings that I am not interested in. Maybe you guys can consider to implement find by one single field, say by the location or the time. In fact, as a user, I would really love to filter the meetings I have yet to complete, so that I can plan my schedule accordingly.

As a user, I would not want to keep referring to the user guide just to remember what fields are optional. While the command result box does provide guidelines after messing up a command, I need to look till the end just to see the (optional) corresponds to which field.

Maybe can consider making it consistent with the user guide by denoting the optional fields in square brackets?

For instance, when I enter 29/02/2022 as the date as below,

the system recorded it as a valid meeting but with the date set as 28/02/2022.

"client" here should be plural.

Full stops are however found in other example usages for other commands, such as in findmeet.

Under findmeet command, I think it should be 'plan discuss matches with discuss plan'.

In my opinion, INDEX is singular, so I think it should be 'INDEX refers...' instead of 'INDEX refer'.

Some of the titles have the first word ending with '-ing', but not for the title for donemeet command.

When deletemeet 99999999999999999999999 is executed, invalid command is thrown.

This is to ensure consistency across the user guide.

Most commands for meeting mode involving index have this problem.

The fact is, nowhere in the user guide forbids user from executing such commands when a client is being shown. The statement with the closest meaning was only mentioned as a tip under findmeet command.

According to the user guide, the findmeet command will return the meetings which has any of the parameters matching the keywords provided, hence I would expect the command to return some meetings when anyone of the above is returned, especially findmeet completed (since the status of the meetings are noted as Completed or Not Completed). However, the system returns no meeting.

One can execute editmeet 3 without a problem. Here, the third meeting is completed.

The meeting status simply switched from completed to not completed.