SharlynLui / pe

0 stars 0 forks source link

Empty page in DG #16

Open SharlynLui opened 5 months ago

SharlynLui commented 5 months ago

Page under Overview is empty, can consider removing this empty page.

image.png

nus-pe-script commented 5 months ago

Team's Response

Thanks for the bug report!

Similar to #2195 which also states about this extra padding issue due to the PDF conversion.

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]