search

vibes-863 / pe

0 stars 0 forks source link

Unclear on what working list is #9

Open vibes-863 opened 5 months ago

vibes-863 commented 5 months ago

image.png

Under the Features section of the UG, there is a feature listed as "Delete a student from the working list".

However, there is no definition of what a working list is and so gets the user very confused.

I listed the severity as High as this affects almost all users who read it.

nus-se-bot commented 5 months ago

Team's Response

We are not sure how the user would get confused.

Right below the sentence, it is mentioned what list the command is referring to. We used working and master list semi interchangeably throughout the user guide regardless.

Nonetheless, there is only one main list that user works with throughout the whole programme, and this is pretty clearly depicted in the UG. Not understanding "what list" should not hinder understanding of the UG much, if not at all. At worst, should be only very minor inconveniences in the legibility of the whole UG due to our choice of words for "working list".

We believe the proposed "high" severity is unwarranted as the UG is not illegible or unusable to the reader as a result of this minor and specific part of the whole guide.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: In the team's response to the bug , they mentioned "We used working and master list semi interchangeably throughout the user guide regardless." I think this does not help their case as I think it leads to more confusion for the reader.

Furthermore, the team accepts that its a bug as they use phrases like "should not hinder understanding of the UG much" (which means it does at least slightly hinder the reader), "At worst, should be only very minor inconveniences".

Thus I think it doesnt make sense that the team rejects this bug as they do agree that it hinders the readers.

## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.High`] - [x] I disagree **Reason for disagreement:** The severity cannot be VeryLow as assigned by the team as it is not a cosmetic bug. In the team response they mention that it is purely cosmetic. It is not to do with the looks of the content but about missing information. This does not fall under the category of cosmetic bugs and hence is not severity of VeryLow