nus-cs2103-AY2223S2 / pe-dev-response

1 stars 0 forks source link

View command does not handle faulty inputs #1890

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

When view is entered with:

  1. no additional parameter, view
  2. non-integer parameter, view abc

No alert message is displayed.

Screenshot 2023-04-14 at 2.37.24 PM.png

I would expect a error message stating that an index is required. For example, something similar to the below delete command error message.

Screenshot 2023-04-14 at 2.37.49 PM.png


[original: nus-cs2103-AY2223S2/pe-interim#1562] [original labels: type.FunctionalityBug severity.High]

Gibson0918 commented 1 year ago

Team's Response

Severity should be medium instead since it does not affect the functionality of the view function, but lowers the user's experience of viewing contact details.

Duplicate status (if any):

Duplicate of #2777