search

seewhyjay / pe

0 stars 0 forks source link

UG doesn't mention that you can swap `eid/` and optional fields for `addEventDetails` and other commands with optional fields #13

Open seewhyjay opened 10 months ago

seewhyjay commented 10 months ago

You can swap the order of eid/ and the other optional fields and the command will still work, same with other commands with optional fields. But this is not mentioned in the UG

image.png

nus-se-script commented 10 months ago

Team's Response

If the user were to swap the positions of the eid and optional fields, they will see no that error message is given and the command functions as expected without hindering the user. Thus, we believe that this is a detail that will not affect the use of the application or the user's understanding of the user guide.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]