TLChicken / pe

0 stars 0 forks source link

Documentation in expel command #17

Open TLChicken opened 3 years ago

TLChicken commented 3 years ago

image.png

There are 2 indexes being referred to, not just 1.

nus-pe-bot commented 3 years ago

Team's Response

Hi, to us the sentence makes sense logically primarily due to the following phrase "shown in the displayed person list and displayed CCA list respectively". Hence, "the index" here will refer to both the indices when we take the whole sentence into account (as readers should).

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: That phrase may make the reader confused because it is referring "the index" to 2 different places (Displayed person list and displayed CCA list), even though there is only ONE index as "the index" implies.

This seems to be a grammatical error as the better way to refer to the indexes is to say "the 2 indexes" instead of "the index".

Since this is a grammar issue that is purely cosmetic, it should be a bug of very low severity.

image.png