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

I can model a pool with MI marker #533

Closed
gimbel opened this issue May 4, 2016 · 10 comments · Fixed by #1393
Closed

I can model a pool with MI marker #533

gimbel opened this issue May 4, 2016 · 10 comments · Fixed by #1393
Assignees
Labels

Comments

@gimbel
Copy link

gimbel commented May 4, 2016

In BPMN it is possible to mark a pool as multi instance (at least the marker looks like the parallel multi instance marker)

Should work for both, collapsed and expanded pools

@gimbel gimbel added modeling good first issue Good for newcomers labels May 4, 2016
@rajgoel
Copy link

rajgoel commented Nov 30, 2017

👍 Would be nice to have this implemented

@nikku
Copy link
Member

nikku commented Nov 30, 2017

We accept contributions for this feature.

@nikku nikku added the pr welcome We rely on a community contribution to improve this. label Feb 19, 2018
@nikku
Copy link
Member

nikku commented Feb 19, 2018

We'll not implement this any time soon, unless we receive a contribution for this feature.

@nikku nikku closed this as completed Feb 19, 2018
@LinqLover
Copy link

Please reopen, I'd love to have this feature, too!
Relevant thread: https://forum.bpmn.io/t/multi-instance-pool/3990/13


I started investigating how the other menu items for pools are implemented and found this line:

actionName: 'replace-with-expanded-pool',

However, neither this repository nor entire GitHub provides a definition of replace-with-expanded-pool. Can you give me any pointer on how and where this symbol is resolved eventually? Conceptionally, I would try to implement a toggle-multiple-instance-pool besides replace-with-expanded-pool, but I have no idea where to start. :-)

@azeghers
Copy link
Contributor

azeghers commented Dec 3, 2020

Hi LinqLover,

Glad to see that you're taking interest in this feature.
If you wish to implement it yourself, I would guide you to this file, where you'll find conditions to add entries to a context menu.
You would then have to create a handler specific to participants, somewhere at the bottom of the file.
Feel free to reach out if you have additional questions.

@azeghers azeghers reopened this Dec 3, 2020
@MaxTru MaxTru added the backlog Queued in backlog label Dec 3, 2020 — with bpmn-io-tasks
@LinqLover
Copy link

Thanks for the pointers. I'll take a look at them when I have more time again. Just to clarify: I'm not actually eager to implement this, someone else would probably be much more efficient at it. :)

Btw: We just found out that bpmn.io also appears not to support multiple instance data object, see https://bpmtips.com/bpmn-miwg-demonstration-2018-nearly-everything-you-always-wanted-to-know-about-data-objects-but-were-afraid-to-ask/#:~:text=collection, is this correct? :-)

@azeghers
Copy link
Contributor

azeghers commented Dec 4, 2020

Understood, I'll take care of it and keep you updated on the results.
As for your side note, we are supporting collection markers for data objects as of v7.5.0 of bpmn-js, so there you go 😄

@nikku
Copy link
Member

nikku commented Dec 5, 2020

capture LfucGg_optimized

@bpmn-io-tasks bpmn-io-tasks bot added in progress Currently worked on and removed backlog Queued in backlog labels Dec 9, 2020
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed in progress Currently worked on labels Dec 9, 2020
@fake-join fake-join bot closed this as completed in #1393 Dec 11, 2020
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Dec 11, 2020
@LinqLover
Copy link

Great, when will these changes arrive at bpmn.io? :-)

@nikku
Copy link
Member

nikku commented Dec 11, 2020

With the next releas 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants