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] Add support for BPMN data objects #2916

Open
StefanAmEnde opened this issue Oct 7, 2023 · 3 comments
Open

[FEAT] Add support for BPMN data objects #2916

StefanAmEnde opened this issue Oct 7, 2023 · 3 comments
Labels
BPMN support Something about the BPMN specification that the lib is already supporting or will support enhancement New feature or request
Milestone

Comments

@StefanAmEnde
Copy link

Describe the bug
When modelling the data-flow perspective in BPMN you use data objects. Although they are correctly shown in the Camunda Modeler, bpmn-visualization will show warnings in the console like e.g.

bpmn-visualization.esm.js:3023 [bv-parser] Edge json deserialization: unable to find bpmn element with id DataOutputAssociation_0yl0bbm

and doesn't display the data objects nor their associations.

To Reproduce
Steps to reproduce the behavior:

  1. Create a bpmn process model containing at least one data-object.
    Example:
    image
  2. integrate the process model in a web application like so:
const xml = "[the bpmn model xml]"
const bpmnVisualization = new BpmnVisualization({container: 'bpmnContainer', navigation: {enabled: true}})
bpmnVisualization.load(xml)
  1. observe that the data object(s) is not displayed and warnings are printed in the console

Expected behavior
I expect that data objects should be displayed.

Screenshots
In camunda modeler:
image

In bpmn-visualization:
image

Console:
image

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser Chrome Version 117.0.5938.134
  • Version 0.41.0

Additional context
I'm using bpmn-visualization in a react application. But the problem should be independent of a specific framework.

@StefanAmEnde StefanAmEnde added the bug Something isn't working label Oct 7, 2023
@tbouffard tbouffard added enhancement New feature or request BPMN support Something about the BPMN specification that the lib is already supporting or will support and removed bug Something isn't working labels Oct 9, 2023
@tbouffard tbouffard changed the title [BUG] BPMN Data Objects are not shown [FEAT] Add support for BPMN data objects Oct 9, 2023
@tbouffard
Copy link
Member

tbouffard commented Oct 9, 2023

Hi @StefanAmEnde and thanks for this detailed feedback.
In short, this is the expected behavior for now, and here's why 👇🏿

We document the BPMN elements that bpmn-visualization supports in https://process-analytics.github.io/bpmn-visualization-js/#supported-bpmn-elements and the documentation sources for the latest version are available at https://github.com/process-analytics/bpmn-visualization-js/blob/v0.41.0/docs/users/bpmn-support.adoc.
As you can read, we don't support data objects at the moment, and there are other types of elements we don't support (some events, all "data" related types, ...).
Here's a milestone that refers to some of these elements: https://github.com/process-analytics/bpmn-visualization-js/milestone/7

About the current issue: I've converted it into a feature request and moved it to the milestone mentioned above.

Are all these explanations clear to you?

@StefanAmEnde
Copy link
Author

Alright. Thanks for the reply👍

@tbouffard
Copy link
Member

tbouffard commented Oct 12, 2023

To complement my first comment:

  • We have no plans to support BPMN data objects in the near future.
  • But this doesn't mean we won't do it later
  • Our decision may be influenced by the number of people requesting this feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BPMN support Something about the BPMN specification that the lib is already supporting or will support enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants