-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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(slack/send-block-kit-message): allow to set unfurl properties in block messages #12003
feat(slack/send-block-kit-message): allow to set unfurl properties in block messages #12003
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Ignored Deployments
|
Thank you so much for submitting this! We've added it to our backlog to review, and our team has been notified. |
Thanks for submitting this PR! When we review PRs, we follow the Pipedream component guidelines. If you're not familiar, here's a quick checklist:
|
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.
Thank you for your contribution! Looks good, ready for QA
Thank you! Did you see my note? Would it make sense to pass through properties by default? |
And one more question: is there an easy way for me to test such changes against my pipedream workspace? |
Hi @joscha . I think it makes sense to pass through the available properties. We'll be making some updates to Slack components in the near future. @malexanderlim This is something to consider for the upcoming Slack Usability Audit. To test changes you make to components, you can use the Pipedream CLI and |
After we merged this PR, I'd open another one that reuses the properties, is that okay?
Is there a special flow I can use in this component, especially because it references multiple files, etc.? The pipedream cli doesn't discover the dependency graph and uploads all needed references, does it? |
Hi everyone, all test cases are passed! Ready for release! |
Now I'm hungry 😋 |
Yes, you can open another PR with the additional changes.
You can |
WHY
Adds
unfurl_media
andunfurl_links
options to slacksend-block-kit-message
action.Background
Contrary to
send-custom-message
which had this functionality for a while:pipedream/components/slack/actions/send-custom-message/send-custom-message.mjs
Lines 36 to 47 in f2ea490
currently, it is not possible to define whether to unfurl links and media in block messages or not.
This adds the two optional properties to the
send-block-kit-message
action.