hungvo0603 / pe

0 stars 0 forks source link

deleterating command accepted when no rating was present #7

Open hungvo0603 opened 3 years ago

hungvo0603 commented 3 years ago

At first, aa_bb got no ratings. Should deleterating command be invalid?

image.png

nus-pe-bot commented 3 years ago

Team's Response

Thank you for pointing this out but as it does not affect the User experience nor does it cause the system to crash whatsoever, I think this bug is of very low severity as it is just a minor exception handle

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: When there is no rating, it is somehow illogical for the program to delete it. Instead, the program should inform the user that there is current no recorded rating. Sometimes it will affect critical users when they find this out.

However, I would change the severity to low because it’s a rare situations and doesn’t greatly affect normal usage..