search

JoelHo / pe

0 stars 0 forks source link

Activity diagram for view person is inappropriate #24

Open JoelHo opened 3 years ago

JoelHo commented 3 years ago

The activity diagram for view person is not very appropriate as it does not really show more information than a numbered list.

image.png

nus-pe-bot commented 3 years ago

Team's Response

We felt that view persons is very similar in implementation to other view commands. Hence, we opted to include a simpler case that summarizes the other view diagrams and their work flow.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: If the implementation is very similar, perhaps the diagram should not have been included at all, instead a hyperlink to a similar flow would be better. This sequence diagram does not show any useful information.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]