-
Notifications
You must be signed in to change notification settings - Fork 0
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: allow async form providers #4
Conversation
a3cc8b2
to
eaa4b0c
Compare
package.json
Outdated
"bpmn-js": "^14.0.0" | ||
"@bpmn-io/form-js": "^1.4.1", | ||
"@bpmn-io/variable-resolver": "^1.2.0", | ||
"bpmn-js": "^15.2.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to make form-js and bpmn-js peer dependencies here? I believe this is one of the places where this is granted.
In fact we don't want to use our own forms + bpmn-js, but resort on whatever is used as part of the stack, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, I'll change it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
==> 7d56c04
I also marked variable-resolver
as peer, since it does not make sense to have form resolving without variable completion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great stuff.
related to https://github.com/camunda/product-hub/issues/1792
closes #3