soc-se-bot
commented
2 years ago Team's Response
Hi! Thank you for bringing up this issue. Since there are two issues stated here, we are responding to the first issue where there are the info is repetitive. From the diagram, we believe that the information is not repetitive and aids the developers in better understanding how this command functions. Furthermore, this diagram is made to give the most information to the developer as a more abstract diagram is provided before this diagram.
Items for the Tester to Verify
:question: Issue response
Team chose [response.Rejected]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
:question: Issue severity
Team chose [severity.VeryLow]
Originally [severity.Low]
- [ ] I disagree
Reason for disagreement: [replace this with your explanation]
Too many uneccessary details shown in sequence diagram, notably the creation of each model class. This could have been encapsulated within a ref. As it stands, it is too much repetitive information, making the diagram unecessarily large and hard to read (text very small).