guanyz / pe

0 stars 0 forks source link

clear deletes aliases #8

Open guanyz opened 3 years ago

guanyz commented 3 years ago

The documentation states that clear only deletes rooms, residents, and issues. However, it also deletes aliases.

Steps to reproduce:

  1. alias a/rf cmd/rfind
  2. clear
  3. aliases. Note that there are no aliases.
nus-pe-bot commented 3 years ago

Team's Response

Severity should be low because:

Type is documentation because:

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: Even if intended, this is still a severity.Medium feature flaw. Unlike app data, user aliases are more like user configurations. Being unable to clear out the app's data without also removing some user configurations fits the bill of severity.Medium, in my opinion, as they cannot bulk-remove data without having to reconfigure all their aliases:

severity.Medium: A flaw that causes occasional inconvenience to some users but they can continue to use the product.