search

timothysashimi / pe

0 stars 0 forks source link

Limit is not specified in use case 11 #8

Open timothysashimi opened 7 months ago

timothysashimi commented 7 months ago

As a reader, i was confused as to what the limit refers to in use case 11. Is it the limit to the number of books the library can contain or another possible limit? Because I don't remember seeing a limit feature of the application other than threshold, which i feel that it should be specified, such as in use case 12. This may cause confusion to the readers and make them assume what the limit refers to. If it is specified in use case 12, it should be standardised and specified in use case 11. Threshold is used instead of limit in another section. I feel that it should be standardised throughout the DG.

image.png

image.png

nus-se-script commented 6 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Use case in DG regarding set limit for library is unclear on what is limit

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

This use case depicts the the library manager being able to set the limit of the Library. However, it is unclear what limit they are referring to. Is it the threshold limit for the merit score, or is it the limit on number of users being able to be added to the library or is it the the limit on the number of books being added to the library.

image.png

[original: nus-cs2103-AY2324S2/pe-interim#3439] [original labels: severity.Low type.DocumentationBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Thank you.

It is in the glossary of UG but not in DG. Good catch.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]