search

JoelHo / pe

0 stars 0 forks source link

Invalid session error unclear #2

Open JoelHo opened 3 years ago

JoelHo commented 3 years ago

Error for invalid session "Session ID should only be c/[session ID], and it should not be blank" is a bit unclear, there is no mention of session id or where I can find it.

Based on the error message, I may try and correct it to c/c/1 since it seems like the ID is "c/1" rather than 1?

image.png

nus-pe-bot commented 3 years ago

Team's Response

It is clarified in the examples and is consistent with the UG.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The UG states,

image.png

which makes no mention of Session ID. Further, some other mentions of "Session ID" in the UG are as follows:

image.png

image.png

It is rather ambiguous what the "Session ID" is referring to, if the error said ID instead it would be ok. In the second screenshot's example, it states that

... details of the session with session ID c/1 on the Left ...

Which can be confusing as I initially interpreted that was "Session ID" = "c/1"