-
Notifications
You must be signed in to change notification settings - Fork 199
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
fix: subscription coorelation key tooltip added #1109
Conversation
b5ae1e4
to
c8ad74e
Compare
Not sure if the message brings the point across, what about the following instead:
|
Yeah, that sounds about right. If we want to add a little more detail (maybe redundant) we can do this
not sure about this though |
I'll keep it short and simple, and go from there. |
As short as I can get it:
|
My critique is that the result is not used, but instead the result creates an
|
Sounds good to me! |
Related to camunda-modeler: #4833
c8ad74e
to
3c4395e
Compare
Good stuff 👏 |
Related to camunda-modeler: #4833
Proposed Changes
Added tooltip for Subscription correlation key
Checklist
To ensure you provided everything we need to look at your PR:
@bpmn-io/sr
toolCloses {LINK_TO_ISSUE}
orRelated to {LINK_TO_ISSUE}