Incorrect link

[liauzhanyi]

click on Logic.java brings me to the wrong repo

[nus-pe-script]

Team's Response

Thank you for raising this issue. We think that this issue is valid. However, we disagree with the severity of the bug.

According to the course website:

Based on the above screenshot, this bug should either be a Low or Medium severity bug. Below is the definition of ‘Low’ and ‘Medium’ severity bug based on the course website:

The purpose of the link & diagram in the DG is for the developer to understand the architecture of those components and understand how one class interacts with the other, not as a source of truth of code (which they should have obtained from our repo).

Based on the above definition, this bug should only be a medium severity bug if:

• The class architecture changes significantly from AB3, causing ‘occasional inconvenience’ to some readers when they open the link
• The way our product’s component interacts with other component differ with how AB3 behaves, which also causing confusion to some readers when they open the link

However, for our case:

• The code architecture for those classes (e.g. Logic.java) is exactly the same as Talentcy’s code. This will not mislead the readers.
• The way Talency’s component interacts with other components are also the same with how AB3 behaves, as seen in both codebases. Again, this will not mislead the readers.
• Not only the link and diagram, but the explanations under that section are already correct and applicable for both AB3 and Talentcy.

Hence, this should not hinder the reader’s understanding of Talentcy’s architecture, which means it should be tagged as severity Low instead of Medium or High.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---