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

Add subprocess with start event included #1242

Closed
2 tasks
volkergersabeck opened this issue Feb 18, 2019 · 2 comments · Fixed by #1404
Closed
2 tasks

Add subprocess with start event included #1242

volkergersabeck opened this issue Feb 18, 2019 · 2 comments · Fixed by #1404
Labels
BPMN enhancement New feature or request good first issue Good for newcomers modeling ux
Milestone
M29

Comments

@volkergersabeck
Copy link

volkergersabeck commented Feb 18, 2019 •
edited by nikku
Loading

Sub-Processes with contents always require a single start event. Right now, the user needs to manually add it every time she creates a new expanded sub-process:

1242 add subprocess with start event

Expected Behavior

  • When adding an expanded subprocess from the palette, always add a start event to it.
  • When transforming a task to an expanded sub-process, add a start event to it

To be discussed: Do we want to extend this behavior, i.e. add the start event for collapsed sub processes, too?
@pinussilvestrus pinussilvestrus added the backlog Queued in backlog label Feb 20, 2019
@nikku nikku added this to the M29 milestone Mar 6, 2019
@nikku nikku added good first issue Good for newcomers enhancement New feature or request labels Mar 6, 2019
@nikku nikku added ready Ready to be worked on and removed backlog Queued in backlog labels Apr 2, 2019
@falko falko mentioned this issue Apr 11, 2019
Open
@gustavjf gustavjf self-assigned this May 14, 2019
@gustavjf gustavjf added in progress Currently worked on and removed ready Ready to be worked on labels May 14, 2019
@gustavjf gustavjf mentioned this issue May 21, 2019
Merged
@nikku
Copy link
Member

nikku commented May 29, 2019

Implemented upstream.

@nikku nikku added ready Ready to be worked on fixed upstream Requires integration of upstream change and removed in progress Currently worked on labels May 29, 2019
@nikku nikku assigned nikku and unassigned gustavjf and nikku Jun 5, 2019
nikku added a commit that referenced this issue Jun 13, 2019
@nikku
feat(client): bump to bpmn-js@4.0.0-beta.1
bc7513d 
Closes #1242
Closes #903
Closes #637
Closes #1290
Closes #609
Closes #608
Closes #1206
Closes #464
Closes #1019
Closes #239
@nikku nikku mentioned this issue Jun 13, 2019
Merged
@bpmn-io-tasks bpmn-io-tasks bot added needs review Review pending and removed ready Ready to be worked on labels Jun 13, 2019
@nikku
Copy link
Member

nikku commented Jun 13, 2019

Closed via #1404.

@nikku nikku closed this as completed Jun 13, 2019
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jun 13, 2019
@nikku nikku removed the fixed upstream Requires integration of upstream change label Jun 13, 2019
@nikku nikku removed the fixed upstream Requires integration of upstream change label Jun 13, 2019
nedyalkonikolov pushed a commit to nedyalkonikolov/camunda-modeler that referenced this issue Jun 18, 2019
@nikku @merge-me
feat(client): bump to bpmn-js@4.0.0-beta.1
f272681 
Closes camunda#1242
Closes camunda#903
Closes camunda#637
Closes camunda#1290
Closes camunda#609
Closes camunda#608
Closes camunda#1206
Closes camunda#464
Closes camunda#1019
Closes camunda#239
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BPMN enhancement New feature or request good first issue Good for newcomers modeling ux
Projects
None yet
Milestone
M29
Development

Successfully merging a pull request may close this issue.

feat(client): bump to bpmn-js@4.0.0-beta.1 camunda/camunda-modeler
4 participants
@nikku @volkergersabeck @gustavjf @pinussilvestrus