-
Notifications
You must be signed in to change notification settings - Fork 72
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
Add BPMN MIWG 2023 demo results to reference #912
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is also a need to modify the test-case-structure.json at the root to add this test case
Reference/C.9.1.png
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This model has a lane that goes over the pool header. This should be corrected.
Reference/C.9.2.bpmn
Outdated
<callActivity name="Document Request" | ||
id="_1c9f4f0d-d193-4643-8201-0843a9a937fe" | ||
isForCompensation="false" | ||
calledElement="process_1c9f4f0d-d193-4643-8201-0843a9a937fe" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The called element here is not defined or imported, there is no way to know if the + marker should be displayed in the diagram.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This notion is still valid in the reference models. I will add an import to fix it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added the imports via 5252a24
@barmac to use the reference models instead |
I've updated the test cases to use the reference models, and added the entries in the JSON. |
Make sense to me, we should merge |
This PR adds the 2023 demo results to the Reference directory. The models used are from the live demo folder, and the renderings were produced with bpmn-to-image, equivalent to Camunda Desktop Modeler rendering.