nus-pe-bot
commented
3 years ago Team's Response
The activity diagram was meant to show a successful delete document operation. However, other diagrams can be included as an extension to the Developer Guide.
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]
The delete operation activity diagram shown below shows the flow of events for a successful delete operation. However, for a more comprehensive diagram, I think it would be better for the diagram to also show how an invalid delete command would be processed so that forks can be added at certain points such as checking if the user index is valid (as shown 4th bubble of the activity diagram). This would make the activity diagram more complete and relevant because it can showcase alternative flows of events, which is one of the main objectives of an activity diagram.