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

Bump to bpmn-js 9.x.x #115

Closed
ps91 opened this issue Mar 4, 2022 · 4 comments
Closed

Bump to bpmn-js 9.x.x #115

ps91 opened this issue Mar 4, 2022 · 4 comments

Comments

@ps91
Copy link

ps91 commented Mar 4, 2022

What should we do?

We need to update token simulation to access the DI trought the utility function getDi or trought the element.

Why should we do it?

Right now the token simulation doesn't work with the newest version of bpmn-js giving this error:

unhandled error in event listener Error: Tried to access di from the businessObject. The di is available through the diagram element only. For more information, see bpmn-io/bpmn-js#1472
@adrienCadoret
Copy link

Hey ps91, got the same error..

@ps91
Copy link
Author

ps91 commented Mar 7, 2022

Yes, we need to update to the newest version of bpmn-js.

@ps91
Copy link
Author

ps91 commented Mar 7, 2022

I noticed there's already a commit fixing this. Can you create a version with this fix?

@nikku
Copy link
Member

nikku commented Mar 9, 2022

Basic support released with v0.27.0.

Follow up, so you are aware of current limitations: #114

@nikku nikku closed this as completed Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@nikku @ps91 @adrienCadoret