-
Notifications
You must be signed in to change notification settings - Fork 401
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
Adding SocketModeReceiver to Bolt-js #630
Conversation
Codecov Report
@@ Coverage Diff @@
## v3 #630 +/- ##
==========================================
- Coverage 83.13% 76.91% -6.23%
==========================================
Files 8 9 +1
Lines 765 849 +84
Branches 230 263 +33
==========================================
+ Hits 636 653 +17
- Misses 85 146 +61
- Partials 44 50 +6
Continue to review full report at Codecov.
|
c833f14
to
1b921ed
Compare
TODO:
|
2dfda3f
to
a4dddce
Compare
dd9b6d7
to
65274df
Compare
Tests are expected to fail right now due to us needing a new release of @socket-mode package |
Co-authored-by: Kazuhiro Sera <ksera@slack-corp.com>
Co-authored-by: Kazuhiro Sera <ksera@slack-corp.com>
Co-authored-by: Shay DeWael <sdewael@slack-corp.com>
aad0674
to
ea3b645
Compare
Summary
In progress! Adding new receiver for socket-mode to bolt-js
Requirements (place an
x
in each[ ]
)