-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Creating Message Flow in Process Not Prevented #1902
Comments
I was considering that, too, but wasn't sure if that would look broken. |
What should happen if some elements are already present on the diagram? How about if the diagram is already a collaboration vs. a process diagram with elements? If we are to create missing participants, should the referred processes be executable or not? Which participant should refer to already existing process? I am not sure about the answers to the questions above. Therefore, I'd suggest to start small with message flows removal. The user can always wrap elements in participant(s), but moving from collaboration to a single process diagram is much more cumbersome. |
I discussed this again with @nikku. We concluded that removing the message flows is the least intrusive option for now. Copying, as shown in the recording, is considered an edge case. We would assume that you either don't copy message flows or you copy the participants as well. We're going to remove the message flows on paste if the participants weren't copied. |
Describe the Bug
The rules don't prevent message flows from being created in processes when multiple elements are created at once. The rules check only if the message flow can connect source and target but doesn't check the parent.
Steps to Reproduce
Expected Behavior
Create is prevented
Environment
The text was updated successfully, but these errors were encountered: