vimuthm / pe

0 stars 0 forks source link

Overzealous input validation #2

Open vimuthm opened 2 years ago

vimuthm commented 2 years ago

Genrelist is too restricted, and a user is unable to add in different genres.

image.png

nus-pe-bot commented 2 years ago

Team's Response

From Team AniList:

image.png

genres

This list covers all of the major anime genres there are. While it is non-exhaustive, this list is more than sufficient to cover almost all animes. We do not consider this as Overzealous input validation as the design choice to include such restrictions was clearly mentioned in the DG. It is to be noted that such a restriction does not drastically impact the overall experience of the user when using this application. The restriction was made to ensure a better experience when using the stats functionality of the application. We also stated that we plan to add the feature of adding new genres in the future in a more careful way to avoid introducing more bugs into the application.

We will be rejecting this bug.

Best Regards!

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]