node-slack-sdk
node-slack-sdk copied to clipboard
fix(1528): better warnings about webclient fallbacks
Summary
fixes https://github.com/slackapi/node-slack-sdk/issues/1528
this PR updates the warning logic for the conversation api to better match this logic:
If the top-level text does not exist and any of its attachments does not have fallback property, we still should warn it (we can say "You should have either top-level text or fallback in all the attachments" if we want to make it more accurate). If all the attachments have the fallback property, no warnings should be printed.
Requirements
- [x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
- [x] I've read and agree to the Code of Conduct.