drustanyjt / pe

0 stars 0 forks source link

Not possible to remove stars #3

Open drustanyjt opened 2 months ago

drustanyjt commented 2 months ago

Feature Flaw Description

The first thing I intuitively wanted to do after adding stars was to find a way to remove stars. The FAQ does mention that this is currently not supported, but this seems like a rather important feature to have.

image.png

Severity

severity.Low : A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only.

Not a purely cosmetic issue in docs or UI, so cannot be VeryLow. While this is an inconvenience since many users would want to unstar, I gave this Low instead of Medium severity since the FAQ has a work around.

nus-pe-script commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

No good way to remove stars/bolts from people

Since star and bolt only take in positive integers, any form of taking away stars/bolts requires the edit command. which can be troublesome if you don't know the student's current star count. This may make it difficult for teachers that want to punish bad behaviour as well as reward them.


[original: nus-cs2103-AY2324S2/pe-interim#2379] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Hi, thanks for the suggestion. Not allowing adding of "negative" stars was our design decision from the start when adding the stars feature, since the inclusion of negative numbers into command may be confusing for some people in our target audience.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]