-
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
Bump @slack/types
and consume event payloads from it
#2223
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2223 +/- ##
=======================================
Coverage 81.59% 81.59%
=======================================
Files 19 19
Lines 1646 1646
Branches 464 464
=======================================
Hits 1343 1343
Misses 194 194
Partials 109 109 ☔ View full report in Codecov by Sentry. |
@slack/types
and consume event payloads from it
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.
Looks good 💯
/** | ||
* A Slack Events API event wrapped in the standard envelope. | ||
* | ||
* This describes the entire JSON-encoded body of a request from Slack's Events API. | ||
*/ | ||
export interface EnvelopedEvent<Event = BasicSlackEvent> extends StringIndexed { |
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 PR removes event payload types from bolt-js, as they were added to
@slack/types
. A few straggling types were left to maintain backwards compatibility.