nus-pe-bot
commented
3 months ago [IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]
Team's Response
Our sincerest apologies.
After serious discussion among the team, we conclude that the severity to be Low.
Reason: The reason why the severity is low is because the actual usage of the tags is not very essential to the functionality of our application. Our app is designed to give a framework to library managers to help manage their libraries, hence we wish to allow for the managers to have more freedom to use our tags as they deem fit. We do want to show that the tags used can be very flexible and hence left the tags to be more free for interpretation.
We do however understand that it could cause a small confusion initially, hence we have accepted the issue as valid.
Items for the Tester to Verify
:question: Issue severity
Team chose [severity.Low]
Originally [severity.Medium]
- [ ] I disagree
Reason for disagreement: [replace this with your reason]
My sincere apologies, i saw that in your UG y'all had a very nice built in tags like "Blacklist" and "Regular"
Which are very relevant to a library/bookself addressbook.
However, the error messages and the default tags used in the "beginning data" are "Owes money" and "friends" "classmates", which are not relevant to a library/bookself addressbook.