Neha-5678 / pe

0 stars 0 forks source link

Error message for deleting a property could be clearer #3

Open Neha-5678 opened 2 years ago

Neha-5678 commented 2 years ago

Screenshot 2021-11-12 at 4.48.43 PM.png The error message could state that the index provided is not positive instead of stating that the command format is invalid so that it is clearer to users.

nus-pe-bot commented 2 years ago

Team's Response

The help message does say that the index must be a positive integer

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Screenshot 2021-11-18 at 6.19.36 PM.png Based on the screenshot above which was taken from the CS2103T website, the error message simply says 'Invalid command format' without giving the reason. I believe that editing the error message will not take much effort as this was done for another feature implemented by the group as shown in the screenshot below. In response to what the team said, the help message only restates the requirement of command format just like what the User Guide does. It still leaves the user to interpret the requirements, check the command typed against the requirements stated and realise that he or she typed an invalid integer/negative integer. Hence, I think a more specific error message should be provided to make the application more user-friendly.

Screenshot 2021-11-18 at 6.24.58 PM.png


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]