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.

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

[FEAT] Render all Conditional Events #792

Conversation

VickyPicky
Copy link
Contributor

closes #651

image
image

@csouchet csouchet changed the title [FEAT] render non interrupting conditional intermediate event attached to an activity boundary [FEAT] Render all Conditional Events Oct 20, 2020
@csouchet csouchet added BPMN rendering Something about the way the lib is rendering BPMN elements depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first enhancement New feature or request external contribution 👤 Pull requests provided by someone who is not a core maintainer rebase needed 💥 Pull request that must be rebased on the latest master commit prior being reviewed or merged labels Oct 20, 2020
Copy link
Member

@csouchet csouchet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rebase needed on https://github.com/VickyPicky/bpmn-visualization-js/tree/317-Detect_Non-interrupting_Conditional_Intermediate_Event_Attached_to_an_Activity_Boundary

A little modification of the documentation is missing, and everything is ok 😄

@@ -326,6 +326,10 @@ The event definition can be defined on the event or on the definitions.
|Signal Non-interrupting Boundary Event
|icon:check-circle-o[]
|The stroke & icon width may be adjusted

|Conditional Non-interrupting Boundary Event
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The documentation for the other Conditional events is missing 😢

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was some issue with merge
I think doc is complete, @csouchet can you confirm please?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, everything is here 👍

@csouchet
Copy link
Member

The final render is OK
image

@csouchet
Copy link
Member

csouchet commented Oct 21, 2020

Rebase from the master branch is needed 😄

@csouchet csouchet removed the depends on another PR ⚠️ Pull request depending on another one. The depending must be merged first label Oct 22, 2020
@aibcmars aibcmars force-pushed the 651-Render_Non-interrupting_Conditional_Intermediate_Event_Attached_to_an_Activity_Boundary branch from 56264fa to ec2f842 Compare October 22, 2020 15:44
@aibcmars aibcmars removed the rebase needed 💥 Pull request that must be rebased on the latest master commit prior being reviewed or merged label Oct 22, 2020
…termediate_Event_Attached_to_an_Activity_Boundary
@csouchet csouchet added the hacktoberfest-accepted Accepted Pull Request during Hacktoberfest label Oct 22, 2020
@csouchet csouchet merged commit bba6007 into process-analytics:master Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BPMN rendering Something about the way the lib is rendering BPMN elements enhancement New feature or request external contribution 👤 Pull requests provided by someone who is not a core maintainer hacktoberfest-accepted Accepted Pull Request during Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Render all Conditional Events
3 participants