tau-bar / pe

0 stars 0 forks source link

Unclear which index is being referred to. #18

Open tau-bar opened 2 years ago

tau-bar commented 2 years ago

Expected:

• Expected Behaviour: To be clear which index is being referred to.

Actual:

• Actual Behaviour: Unclear as to which index is being referred to, when UG states at the specified index, as there is PATIENT_INDEX and APMT_INDEX.

• Why that severity? Might cause some inconvenience for new users reading UG.

Steps to reproduce:

  1. Go to edit an appointment section.

Screenshots:

Screenshot 2021-11-12 at 5.27.20 PM.png

nus-pe-bot commented 2 years ago

Team's Response

"Unclear as to which index is being referred to, when UG states at the specified index, as there is PATIENT_INDEX and APMT_INDEX."

We find that this is untrue.

Screenshot 2021-11-13 at 1.58.27 PM.png

We are clearly referring to the APMT_INDEX in this sentence, pointed out by the tester. There is no mention of "patient" in the sentence for any reasonably careful reader to interpret otherwise.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: ### Response

I disagree. It does cause inconvenience to the users reading this sentence.

Rationale

The sentence could still be unclear to readers. If the writers had written 'at the specified appointment index', it would be clear. But the use of multiple parameters with the word INDEX in them could cause confusion to the reader. For those features where there is only one index parameter, it is clear to the reader, eg:
Screenshot 2021-11-17 at 7.56.36 PM.png

However the one I highlighted during the PE can cause minor inconvenience to the user's reading experience, and therefore, this is classified as a severity.Low bug:
Screenshot 2021-11-17 at 7.59.03 PM.png