refactor(slack): move common props to common/send-message and remove superfluous fields
WHY
Currently, there are some common props that are duplicated in all Slack components that send messages in any way (direct messages, group messages, private messages, thread responses, etc.
This pull request moves the two new properties introduced in #12003 to the common props. It also removes duplicated properties already defined in the common props that are splatted into each of the components depending on it.
cc @michelle0927
The latest updates on your projects. Learn more about Vercel for Git ↗︎
2 Ignored Deployments
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| docs-v2 | ⬜️ Ignored (Inspect) | Visit Preview | May 22, 2024 6:22am | |
| pipedream-docs-redirect-do-not-edit | ⬜️ Ignored (Inspect) | May 22, 2024 6:22am |
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:
- [ ] Create components to address specific use cases whenever possible
- [ ] Component
keys should follow the formatapp_name_slug-slugified-component-name - [ ] Components should follow the standard directory structure
- [ ] Prefer Node.js client libraries to REST APIs
- [ ] When making API requests, handle pagination to ensure all data / events are processed
- [ ] Use
secretprops to capture sensitive data - [ ] Props and methods should be defined in app files whenever possible
- [ ] Document methods with JS Docs
- [ ] Use
optionalprops whenever possible, and set adefaultvalue where you can - [ ] Use async options to accept user input wherever possible
@vunguyenhung 👋 is there anything left to do for me on this pull request?
Hey @joscha, nothing else! This should be good to release!
Hi everyone, all test cases are passed! Ready for release!
Test report https://vunguyenhung.notion.site/refactor-slack-move-common-props-to-common-send-message-and-remove-superfluous-fields-12018-ebbf5baadbce4c00a5a5cd7fb3611d05