eugenetyc / pe

0 stars 0 forks source link

Error message for editp should include that at least one parameter should be provided. #8

Open eugenetyc opened 4 years ago

eugenetyc commented 4 years ago

Seen below is the error message for editp 1. We should not assume users understand why they are wrong (in this case as they didnt enter the minimal number of parameters).

nus-pe-bot commented 4 years ago

Team's Response

As clearly stated in the error message, the parameters are INDEX (must be a positive integer) [d/DESCRIPTION] [cp/COST PRICE] [pr/PRICE] [q/QUANTITY] [s/SALES]. In order to edit a product, it is intuitive that one of the fields would be required otherwise there is no point in using the command since no modified fields are given.

Moreover, in our UG, it is stated that one of the optional fields are required.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[ ] I disagree

Reason for disagreement: [replace this with your reason]