nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Command flag order is important despite UG description. #1722

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

UG states that command flags are order-insensitive. image.png

However, entering


[original: nus-cs2103-AY1920S1/pe-interim#1731]

justarock111 commented 4 years ago

Team's Response

Entering the command that was supposed to fail(as highlighted by tester) work fine, with the feedback and result displayed on GUI being correct. We were unable to reproduce on any of our computers, and we have tested this countless number of times so we know it's supposed to work. So we have marked our response as Rejected.

We suspect that the user may have run the two commands back to back without deleting the mentor. This would cause an exception saying that the mentor trying to be added is already present in Alfred. However, we are not sure if this is the issue because no pictures of the response message were attached.

Screenshot 2019-11-17 at 3.17.54 PM.png

Duplicate status (if any):

--