feat: add workflow discovery and execution#40
Conversation
649743a to
6231744
Compare
|
Thx for the PR. Hopefully either one of me or @nwparker can help get it in today. |
969f54c to
3e5d200
Compare
|
Looks like I pushed this with the wrong base last time 🙈 have rebased onto |
|
I like the overall direction here. The one part I’m less sure about is form submission via Would you be open to scoping this PR to:
and then doing form submission as a follow-up once we’ve had a chance to prove out exactly what we want to support there? That feels like a very easy version to land, still adds real value, and keeps us from overcommitting on workflow-form support before we know how robust we want it to be. |
|
@AmethystLiang I like your suggestion to narrow the scope of a single PR, splitting the concerns. I'll take a look into how to split it neatly, if that's doable along the lines you're asking about. I don't think any other slack tool I've seen has managed to successfully do the form submission style of slack workflow, so adding this functionality would give this cli a unique feature. |
3e5d200 to
a17585e
Compare
|
@AmethystLiang have split the code |
97359c7 to
37184b8
Compare
Add `workflow` command with subcommands: - `list` — discover workflows bookmarked in a channel - `preview` — get trigger metadata - `get` — get workflow definition including form fields - `run` — trip a workflow trigger
37184b8 to
e1c8c4d
Compare
Summary
workflowcommand withlist,preview,get, andrunsubcommandsForm submission via
--field(RTM WebSocket) will follow in a separate PR.New files
src/cli/workflow-command.ts— CLI command registrationsrc/slack/workflows.ts— workflow discovery, preview, schema, and trigger APIsUsage
Test plan