Wincenttjoi / CS2103T-PracticalExam

0 stars 0 forks source link

Feature flaws #7

Open Wincenttjoi opened 3 years ago

Wincenttjoi commented 3 years ago

Consolidating all the feature flaws into 1 issue:

nus-se-bot commented 3 years ago

Team's Response

Hi, thanks for your report! Firstly, we believe resetting statistics defeats the purpose of the statistics feature. Statistics is meant to help a user keep track of their learning over time. If a user can keep resetting statistics, they can keep resetting it to maintain a good score. This prevents them from learning effectively and the statistics feature would no longer serve its purpose.

Secondly, sort reviewed is meant to sort flashcards by their review frequency in quiz mode, to allow users to see which flashcards they have reviewed most/least. The user can then know about which flashcards they have not quizzed themselves much.

sort reviewed -a -d will sort flashcards by review frequency in descending order. To clarify, this example is meant to highlight that our app only takes the last instance of the flag.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I find your argument invalid. A user who CAN reset the statistics does not mean that the user MUST reset the statistics. Again, leaving this feature out is a major turn off for users as they cannot track their most recent capability in certain subjects. If you are good at a topic 2 months ago, and does not revise at all until now, there is no guarantee that you're still good at it. Without this reset statistics feature, you are forced to follow what you know/ dont know from many months back. If a user wants to reset to maintain a good score then thats the user's issue and not your application's problem anymore. I'm just suggesting that this feature is good to have.

I understand what does a sort review do, but I can't understand the VALUE that this feature brings out. It is a plain question mark for a user wanting to sort and see how many times they reviewed a question?


:question: Issue severity

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

Reason for disagreement: Frankly speaking, the bug severity given as a guideline only shows the functionality bug but not feature flaw. I'll leave this to the teaching team to decide.