miljyy / pe

0 stars 0 forks source link

Error message shown but command still goes through #7

Open miljyy opened 10 months ago

miljyy commented 10 months ago

Under adding venue to event, viewEvent eid/3 followed by addEventDetails eid/3 pid/1 pid/2 pid/3, then addEventDetails eid/3 pid/2 vne/3 shows an error message in the result display as well as red text in the command box. Despite the red text which suggests an error, the venue still changes accordingly to vne/3, which might be confusing for users.

image.png image.png

nus-se-script commented 10 months ago

Team's Response

The message in the result box as well as the red text informs the user to show that the user added a repeated user. However, since the user did try to add a valid venue to the event. We allow this command to succeed and thus "the error message" shown does not include adding the venue to be unsuccessful. This is working as intended.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]