search

ChocoRagdoll / pe

0 stars 0 forks source link

Problematic error message for all commands with indexes as parameter #6

Open ChocoRagdoll opened 4 months ago

ChocoRagdoll commented 4 months ago

image.png

image.png

When a very large positive integer is entered as the index, the app shows a different error message from the supposed 'the person index provided is invalid'. Since it is rather a rare case for a user to input such a large number, this bug should not be so severe. However, since this happens for every command that includes index, I will tag this as medium.

nus-pe-bot commented 4 months ago

Team's Response

Hi, thanks for your feedback. We have included “Errors Improvement” in our planned enhancements, which mentions this exact error and our plans for improving the consistency of errors. When we execute the planned enhancements, this issue would be resolved.

telegram-cloud-photo-size-5-6136195526866093556-y.jpg

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

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