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.

Sign up for GitHub

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

Jump to bottom

Alternative path not closed in activity diagram for hide #7

Open
chengda300 opened this issue Nov 11, 2022 · 1 comment
Open

Alternative path not closed in activity diagram for hide #7

chengda300 opened this issue Nov 11, 2022 · 1 comment
Labels
severity.Low status.Done type.DocumentationBug

Comments

@chengda300
Copy link
Owner

chengda300 commented Nov 11, 2022 •
edited
Loading

image.png

Opened 2 alternative paths, forget to close them
@nus-pe-script
Copy link

nus-pe-script commented Nov 14, 2022 •
edited by chengda300
Loading

Team's Response

image.png

We think that the merge node is not necessary if the diagram is not ambiguous. Refer to the section at the bottom of the screenshot.

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.Low]

  • 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
Assignees
No one assigned
Labels
severity.Low status.Done type.DocumentationBug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nus-pe-script @chengda300