search

tiffanyliu0220 / pe

0 stars 0 forks source link

Empty Page in the middle of DG #11

Open tiffanyliu0220 opened 7 months ago

tiffanyliu0220 commented 7 months ago

There is an empty page in the middle of DG.

image.png

nus-pe-bot commented 7 months ago

Team's Response

Thanks for pointing this out!

This similar to #2195 which also stated about this extra padding in the DG on page 7.

The 'Original' Bug

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

Missing page

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.

Screenshot 2024-04-19 at 5.35.40 PM.png

Blank page, possible missing diagram

[original: nus-cs2113-AY2324S2/pe-interim#1968] [original labels: type.DocumentationBug severity.High]

Their Response to the 'Original' Bug

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

Thanks for pointing this out!

This blank page (ie too much padding) is most probably as a result from the pdf conversion.

We will accept this as a documentationBug of severity.veryLow as it is a cosmetic issue (ie sequence diagram split between 2 pages) as quoted from the textbook (see below).

image.png

In addition, there is no missing diagram if you see the DG on GitHub.

What is seen on GitHub:

image.png

What is seen on DG:

image.png

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.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]