timothysashimi / pe

0 stars 0 forks source link

Duplicate section in UG #7

Open timothysashimi opened 5 months ago

timothysashimi commented 5 months ago

In the UG, there are these segments. If the app is meant to keep track of books that library users borrow, why is there another section that explains why a "non-library user" cannot borrow a book? I would assume that it is a non library user for the first part, as it's not specified that the error message is for a library user.

image.png

image.png

soc-pe-bot commented 5 months ago

Team's Response

Thank you for reporting this bug.

The first section is from the library's perspective, while the second section is from the Library user's perspective.

The first section is about the borrow and delete commands specifically and how to deal with them when they cannot be used. It is not from the perspective of any library user but rather to help the reader better understand the command itself.

The second section is from the library user's perspective when the user faces issues borrowing a book.

What is most important to note is that the two sections cover different use cases despite their overlaps. Thus we made 2 different sections for them.

Admittedly, we can improve the description in the future to help the reader clearly understand the difference between the 2 sections.

image.png

Note: While response.NotInScope are not counted against the dev team, they can earn a small amount of consolation marks for the tester. The other three do not affect marks of either the dev team or the tester.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]