uggi121 / pe

0 stars 0 forks source link

Edit Command Feature Flaw #3

Open uggi121 opened 5 years ago

uggi121 commented 5 years ago

The edit cheat-sheet command doesn't allow users to edit tags. The only thing that can be done is remove existing tags by typing the tags out verbatim. New tags cannot be added!

In the screenshots below, I try to edit an existing cheat sheet. The edit command displays a successful message, but there is no change. There is no way to add new tags!

image.png

image.png

image.png

nus-pe-bot commented 4 years ago

Team's Response

It is already stated in UG that all optional fields, apart from t/TITLE, will remove the respective content. Furthermore, it also stated that invalid tag/TAG specified will be ignored (which is highlighted in bold).

Important section of edit command of cheatsheet specifying conditions mentioned above: image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The feature works as expected. The primary issue here is the sheer futility of the feature due to reasons elaborated in the initial issue. This is why the bug has been marked as a "Feature Flaw" as opposed to a "Functionality Bug".

The developers have NOT addressed my initial bug. Instead, they've gone on to show that the feature works as expected (which was never the issue in the first place).

This is a fairly major acceptance testing issue, not a system/functional one.

To quote the CS2103T website:

type.FeatureFlaw: Some functionality missing from a feature delivered in v1.4 in a way that the feature becomes less useful to the intended target user for normal usage. i.e., the feature is not 'complete'. In other words, an acceptance testing bug that falls within the scope of v1.4 features. These issues are counted against the 'depth and completeness' of the feature


:question: Issue severity

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

Reason for disagreement: I don't know why the team decided to increase the severity! Nevertheless, I think it should be medium and NOT high, which is probably a mistake on their part.