Remove obsolete missing RTM API support from FAQ#361
Remove obsolete missing RTM API support from FAQ#361aoberoi merged 2 commits intoslackapi:masterfrom
Conversation
|
|
docs-src/faq.rst
Outdated
|
|
||
| On the other hand, |product_name| does not provide access to the RTM or Events bot-building APIs, although we anticipate | ||
| adding support for these in the future. | ||
| On the other hand, |product_name| does not provide access to the Events bot-building API, although we anticipate adding |
There was a problem hiding this comment.
Maybe it would be good to add a reference to the official Events API package, now that its built?
There was a problem hiding this comment.
So besides the RTM being now supported, the Events API is now supported too?
There was a problem hiding this comment.
its not supported with this pacakge, but the slackeventsapi package is another official one that people might want to know about.
There was a problem hiding this comment.
|
🤔 I wonder why the tests are failing here but are okay on |
|
Most probably because they were failing on master when I branched off. I could do a rebase to fix them. |
|
Actually, even |
Codecov Report
@@ Coverage Diff @@
## master #361 +/- ##
==========================================
- Coverage 76.38% 75.66% -0.73%
==========================================
Files 10 10
Lines 415 415
Branches 74 73 -1
==========================================
- Hits 317 314 -3
- Misses 87 90 +3
Partials 11 11
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #361 +/- ##
=======================================
Coverage 76.38% 76.38%
=======================================
Files 10 10
Lines 415 415
Branches 74 74
=======================================
Hits 317 317
Misses 87 87
Partials 11 11Continue to review full report at Codecov.
|
|
Don't know if I chose the right wording though. |
|
That was my plan too, will do later. |
|
@aoberoi, Rebased it. |
|
🤔 Not sure why appveyor is having trouble reporting its status check, but since its not relevant to these changes, i'm fine using my admin rights to merge this. thanks @revolter for sticking with this! |
|
Sorry for the delay though! |
Remove obsolete missing RTM API support from FAQ
Remove obsolete missing RTM API support from FAQ
4 participants
Summary
Fixes #360.
Requirements (place an
xin each[ ])