[PE-D] Day index out of bounds for edit-itinerary-attraction command seems to not throw an error #246
Assignees
Comments
peter-yeh
added a commit
to peter-yeh/tp
that referenced
this issue
Nov 1, 2020
Invalid day will now throw error and highlight the command in red. Closes AY2021S1-CS2103T-T09-3#237, AY2021S1-CS2103T-T09-3#246, AY2021S1-CS2103T-T09-3#236
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When the day index is out of bounds for edit-itinerary-attraction command, there seems to be no error thrown.
In the given screenshot, there are 381 days in the selected itinerary, so day/400 is out of bounds. But instead of throwing an error, when the command in the screenshot was executed, the previous display message continues to stay on in the result box and nothing happens in the app, instead of showing an error message to inform the user what went wrong.
Labels:
severity.Lowtype.FunctionalityBugoriginal: shawn-nyk/ped#7
The text was updated successfully, but these errors were encountered: