Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Potential DG Bug: Diagram contradiction #7

Open
TomLBZ opened this issue Nov 13, 2020 · 1 comment
Open

Potential DG Bug: Diagram contradiction #7

TomLBZ opened this issue Nov 13, 2020 · 1 comment

Comments

@TomLBZ
Copy link
Owner

TomLBZ commented Nov 13, 2020

The diagram above shows that Global connects to other classes such as Apps only through the PlanNUS main class while the bottom diagram shows that Global are connected to those classes Directly, which is contractory and confusing.

image.png

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 18, 2020

Team's Response

image.png

As clearly mentioned in our DG, each of these diagram only shows the connection to a specific class rather than all the connection to every single class. This is done to prevent the diagram from being too complicated. Furthermore, there is no mistake in the format being used for these diagrams as this is a architectural diagram that exist in the "Design" section to show how the different components are connected to each other. We were also careful to not call them class diagrams.

Tagging this 'bug' as a HIGH serverity is clearly overstating the issue as

  1. There is no specific format for architectural diagram
  2. It is clearly stated in the DG the reasons for going with such a layout.

Due to the given reasons above, the team will be rejecting the bug.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement: [replace this with your explanation]


❓ Issue severity

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

  • I disagree

Reason for disagreement: [replace this with your explanation]


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants