search

kokerinks / pe

0 stars 0 forks source link

DG: Planned Enhancements hyperlinks do not work #15

Open kokerinks opened 3 months ago

kokerinks commented 3 months ago

Under Planned Enhancements -> Point 1. (Automatically update team in contacts list when delete-contact is run)

image.png

Both hyperlinks shown and deleting do not work, and lead to http://127.0.0.1:8080/UserGuide.html#list-members-belonging-to-a-team and http://127.0.0.1:8080/UserGuide.html#deleting-a-contact-from-a-team-delete-contact respectively

soc-pe-bot commented 3 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]

DG: Hyperlink does not work

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.

Under Getting Started -> Minimum Requirements: image.png

The hyperlink under this section does not work. it leads to the URL: http://127.0.0.1:8080/UserGuide.html#installation-guide

as shown:

image.png

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

Their Response to the 'Original' Bug

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

image.png

According to the CS2103T website, only content within the PDF files should be considered. Since these links are meant to lead to websites outside of our PDF DG, it is not in scope of Phase II of the PE.

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: image.png

The bug identified as the original is a different hyperlink from the one identified above. Hence, it is not a duplicate.

## :question: Issue response Team chose [`response.Rejected`] - [x] I disagree **Reason for disagreement:** Yes, the link leads to content outside of the DG. Despite that, the **link itself is within the DG**, and **leads the user to a broken URI**, which can be confusing for users. Hence it still be considered as an acceptable bug.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [x] I disagree **Reason for disagreement:** As stated: ![image.png](https://raw.githubusercontent.com/kokerinks/pe/main/files/fb8658e5-5d97-48ed-a147-a5046311437c.png) Also: ![image.png](https://raw.githubusercontent.com/kokerinks/pe/main/files/d113a1c5-f711-41d5-8ff7-efbc4fb51f03.png) Hence, broken links are considered `severity.Low` or `severity.Medium`. It is not a cosmetic bug (and hence not `severity.VeryLow`)