search

rohit0718 / pe

0 stars 0 forks source link

Unintuive delete lesson command #9

Open rohit0718 opened 2 years ago

rohit0718 commented 2 years ago

Currently the commands for deleting lessons is given by deletelesson, which is against the single word commands for everything else. It is likely that the user will type delete lesson instead.

nus-pe-bot commented 2 years ago

Team's Response

The team consensus is to make a clear differentiation between deleting tutee and deleting lessons. Using deletelesson as our command word actually helps to notify our user if they get confused between these 2, and the error message "invalid command" will be thrown if the user accidentally types delete lesson instead.

which is against the single word commands for everything

addlesson is another example of two-word command that we implemented for the same reason, this helps to differentiate between the invalid input add lesson and addlesson.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]