search

jtooya / pe

0 stars 0 forks source link

Links lead to the SE-EDU AB3 implementation of MainWindow and MainWindow.fxml in the DG #3

Open jtooya opened 1 week ago

jtooya commented 1 week ago

Given that you have changed the implementation of the UI, I think that the links provided in the MainWindow.fxml button showed here should route to your implementation, rather than SE-EDU's

Screenshot 2024-11-15 161853.png

image.png

EDIT: multiple links lead to the AB3 implementation, including the Logic link here. This is not a full check of the DG, although I suspect other links may have the same issue.

image.png

nus-se-script commented 1 week 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: Outdated Nagivation Link

In the architecture section, clicking on the Main and MainApp brings me to the github codebase of addressbook.

Screenshots:

image.png

image.png

[original: nus-cs2103-AY2425S1/pe-interim#2350] [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`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]