-
Notifications
You must be signed in to change notification settings - Fork 394
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
Bradh workflow extension support #546
Bradh workflow extension support #546
Conversation
Codecov Report
@@ Coverage Diff @@
## feat-workflow-steps #546 +/- ##
====================================================
Coverage 83.13% 83.13%
====================================================
Files 7 7
Lines 593 593
Branches 184 184
====================================================
Hits 493 493
Misses 68 68
Partials 32 32
Continue to review full report at Codecov.
|
trigger_id: string; | ||
user: { | ||
id: string; | ||
name: string; |
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.
user
object in action payload has a key of username
rather than name
action_ts: string; // undocumented | ||
workflow_step: { | ||
workflow_id: string; | ||
step_id: string; |
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.
workflow_step
also has keys for inputs
and outputs
in either case (add vs. edit)
src/types/events/base-events.ts
Outdated
callback_id: string; | ||
workflow_step: { | ||
workflow_step_execute_id: string; | ||
workflow_id: string, |
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.
A few commas here instead of semi-colons
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.
Left a few suggestions
trigger_id: string; | ||
user: { | ||
id: string; | ||
username: string; |
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.
This was changed from name
to username
based on the payloads
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.
Thanks for catching and fixes all of those 😄
Summary
This PR adds support for a new feature entering open beta called Workflow Steps from Apps. It allows Slack apps to provide custom steps for Slack Workflows. This new feature requires the following updates to Bolt, encompassed in these changes:
workflow_step_edit
actionworkflow_step_execute
eventsay
function from theworkflow_step_edit
handlergetTypeAndConversation()
to allowworkflow_step_edit
action to fall into the action handling logicRequirements (place an
x
in each[ ]
)