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 when removing tasks/events #134

Open
nus-se-pr-bot opened this issue Apr 6, 2018 · 1 comment
Open

Inconsistent error message when removing tasks/events #134

nus-se-pr-bot opened this issue Apr 6, 2018 · 1 comment
Labels
type.bug Something isn't working

Comments

@nus-se-pr-bot
Copy link

As a CS Student, 0 is a positive integer. However feel free to ignore this if it is expected behavior
Expected: Same error message when removing an event that does not exist.

Actual: Error message when index is greater than number of events is
screen shot 2018-04-06 at 4 23 30 pm

And the error message when index is 0 is
screen shot 2018-04-06 at 4 24 44 pm

[original: nus-cs2103-AY1718S2/pe-round1#119]

Issue created by: @davidten

@Kyomian Kyomian added the type.bug Something isn't working label Apr 7, 2018
@Kyomian
Copy link

Kyomian commented Apr 7, 2018

0 is NOT a positive integer; it is a non-negative integer, but I get your point Mr David

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type.bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants