marcus-ny / pe

0 stars 0 forks source link

Error message for deleting beyond the number of existing contacts could be more descriptive. #5

Open marcus-ny opened 7 months ago

marcus-ny commented 7 months ago

Description: Error message for deleting out of bounds (deleting index 10 while list only has 6 contacts for example) can be more descriptive to prompt the user to correct their mistake and enter a number within bounds. Perhaps, a message such as "The number you have entered is beyond the number of contacts in the list. The index must be below (current max index)!" would be more descriptive

Preconditions: Contact list has only 6 people.

Steps to reproduce:

  1. Delete 10

Screenshots: Screenshot 2024-04-19 at 4.30.56 PM.png

nus-se-bot commented 7 months ago

Team's Response

This bug is not in scope as even though the error message can be more descriptive, we do feel that the error message is sufficient for the user to find the issue or troubleshoot as necessary. Moreover, in the UG we clearly state the constraints of this field. The reasons for that error message can also be traced back from the UG, and the client size limit can be inferred that way.

It could be a change that we decide to add in the future, to have clearer error messages.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]