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(palette): allow trigger with entry id #741

Merged
merged 1 commit into from
Jan 24, 2023
Merged

Conversation

smbea
Copy link
Contributor

@smbea smbea commented Jan 24, 2023

In order to support bpmn-io/bpmn-js#1814; similar to how we did for context pad #719

I added some general test cases for #trigger similar to the ones we have for context pad since we didn't have these for palette

Allows manual triggering of entries.
@bpmn-io-tasks bpmn-io-tasks bot added the needs review Review pending label Jan 24, 2023
@smbea smbea requested review from a team, philippfromme and barmac and removed request for a team January 24, 2023 08:48
Copy link
Member

@nikku nikku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff.

@smbea
Copy link
Contributor Author

smbea commented Jan 24, 2023

You know it's a good PR when you get two approvals 😂

@smbea smbea merged commit 3d274e9 into develop Jan 24, 2023
@smbea smbea deleted the trigger-palette-with-id branch January 24, 2023 09:07
@bpmn-io-tasks bpmn-io-tasks bot removed the needs review Review pending label Jan 24, 2023
@nikku
Copy link
Member

nikku commented Jan 24, 2023

You got mine because of

I added some general test cases for #trigger similar to the ones we have for context pad since we didn't have these for palette

image

Not sure why you got @barmac's 😉

@barmac
Copy link
Member

barmac commented Jan 24, 2023

This makes sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants