Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent error message for deletion #9

Open
wilsonwid opened this issue Apr 19, 2024 · 1 comment
Open

Inconsistent error message for deletion #9

wilsonwid opened this issue Apr 19, 2024 · 1 comment

Comments

@wilsonwid
Copy link
Owner

wilsonwid commented Apr 19, 2024

image.png

image.png

There are inconsistent error messages for deletion, as even though 0 and 3 are both invalid indices, delete 0 and delete 3 yield two different error messages.

@nus-pe-script
Copy link

nus-pe-script commented Apr 22, 2024

Team's Response

Thank you for reporting this bug.

The error for delete 0 is invalid command format because INDEX must be a positive integer based on our UG:

Screenshot 2024-04-21 at 15.17.37.png

In the other case, the command delete 3 points to an non-exisiting person in the app. Therefore, the error messages are different.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants