godjuansan / pe

0 stars 0 forks source link

`Sort` ascending #14

Open godjuansan opened 3 years ago

godjuansan commented 3 years ago

sort ascending allows me to sort ascending when the order is already ascending. A message stating that the list is already ascending should be popped out instead.

nus-pe-bot commented 3 years ago

Team's Response

Reason for rejection : A list that is already sorted in ascending order can most certainly be sorted in ascending order again. (Just that no change will occur) This behaviour does not contradict what is stated in the user guide and it is unclear how this behaviour will hinder the user from using the software. Thus it seems to me that you are simply suggesting a quality-of-life change to the product.

Reason for downgrading severity to very low: This behaviour does not hinder the user from using the app.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The user should know if this is already sorted ascendingly. The reason is that this behavior should not be done multiple times as it might confuse the user (especially in huge number of contacts cases) whether the contact is already sorted or not


:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: [replace this with your explanation]