search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

Add command does not allow for remark prefix or changing of remark #1607

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

As a fast typist, I might want to quickly add a remark as I enter the details of the user. Currently, to do this I need to enter 2 separate commands, add then remark. Instead of just using a r/ prefix along with my add command.

image.png

[original: nus-cs2103-AY2021S2/pe-interim#1607] [original labels: severity.Medium type.FeatureFlaw]

IceBear789 commented 3 years ago

Team's Response

Reason for rejection: According to my interpretation of the bug report, the tester seems to be suggesting to add even more parameters to the add and edit commands. However, observe that these two commands are already very long in terms of potential input. (Add command has 5 compulsory parameters and 1 optional parameter) Adding more parameters to the add command will make it even more complicated to use, especially for beginners. (For example, if the user input is long and has a missing backslash ("/") for one of the parameters somewhere, it will be quite difficult to spot the error and correct the input.)

Thus I have decided to separate the remark command from the existing add and edit commands as I do not believe that overloading the add command with more and more parameters is the correct design decision as it decreases usability.

Reason for downgrading severity to low: Even if the change suggested by the tester is implemented, only very experienced users will benefit from the increase in speed. Since the input format for the remark command is already very short, even these experienced users will not get a significant increase in speed.

Duplicate status (if any):

--