chuababyy / pe

0 stars 0 forks source link

Edit will clear all the tags, which is unintuitive #8

Open chuababyy opened 1 year ago

chuababyy commented 1 year ago

Screenshot 2023-11-17 at 4.39.16 PM.png Screenshot 2023-11-17 at 4.39.16 PM

This is stated in the UG, and it is also the behaviour in the application.

However, this is unintuitive, and does not serve the purpose of edit. Because edit command should not clear the previous information. Hence, this should be a feature flaw

soc-se-bot commented 1 year ago

Team's Response

Thanks for your report, but we don't think this should be in scope as it's not important enough a feature to be implemented in 1.4.

While edit should be used to edit the parameters of a member, this feature actually allows them to clear a member's tag while editing other parameters of a member, which can be considered as a convenience for now as they only need to enter 1 command.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: I disagree that this is NotInScope

When you implement a feature called edit, it should give the users the ability to edit what they want, and only what they want.

If you have a feature called edit, but instead it clears the current information (then updates it with the new information), then this is not only unintuitive, but also doesnt serve the purpose of the function. Therefore, I believe this is a FeatureFlaw, and should be addressed in v1.4.