vigonometry / pe

0 stars 0 forks source link

[Developer Guide] Links to the MainWindow class and component point to the SE-EDU repository #8

Open vigonometry opened 1 year ago

vigonometry commented 1 year ago

Details

On clicking the links to MainWindow and MainWindow.fxml in the UI component section, the website provided is SE-EDU

To reproduce

Click on links in the UI Component section of the DG

Expected behavior

Points to team's repository here: F11-1

Screenshots / Screen recordings

Screenshot 2023-04-14 at 3.15.16 PM.png

nus-pe-script commented 1 year ago

Team's Response

I do not think it is medium priority. It should be low since it is inferred that it is our team. Please change it to low, then I will accept it.

The 'Original' Bug

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

MainWindow and MainWindow.fxml leads to the AB3 MainWindow and MainWindow.fxml not your group's one

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.


image.png

image.png

image.png I'll consider it low since it does not hinder the user too much based on cs2103 website


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

Their Response to the 'Original' Bug

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

No details provided by team.

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.Medium`] - [x] I disagree **Reason for disagreement:** ![Screenshot 2023-04-18 at 12.08.17 PM.png](https://raw.githubusercontent.com/vigonometry/pe/main/files/0d257b81-a9a2-4a9e-b701-35d06dc0e612.png) - Based on the website's definition a medium-severity bug is one that causes an occasional inconvenience to the user but can continue to use their products. The wrong links to `MainWindow` and `MainWindow.fxml` cause occasional inconvenience to developers attempting to improve these particular features. Furthermore, having made extensive UI changes to the design, redirecting to the SE-EDU's page, might make it appear that, the code presented here is a representation of their UI, which may result in confusion on the part of the developer. Also, this should not be left up to the "inference of readers" as the developer puts it since the Developer would need to click on working links, navigate to their repository and find these components which causes an inconvenience.