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 View API's and fixing OAuth Headers Bug #517
Conversation
Codecov Report
@@ Coverage Diff @@
## master #517 +/- ##
==========================================
- Coverage 69.23% 68.89% -0.34%
==========================================
Files 15 15
Lines 1690 1707 +17
Branches 93 96 +3
==========================================
+ Hits 1170 1176 +6
- Misses 498 508 +10
- Partials 22 23 +1
Continue to review full report at Codecov.
|
|
||
See `views.update <https://api.slack.com/methods/views.update>`_ for more info. | ||
|
||
If you want to push a new view onto the modal instead of updating an existing view, reference the `views.push <https://api.slack.com/methods/views.push>`_ documentation. |
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.
May be worth mentioning the 3 modal limit here
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.
We have a subsequent docs revamp that will be shipped as a fast follow. I'll be sure to update it there.
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.
Approved pending testing this afternoon 😎
Summary
This PR adds new View API's.
It also fixes a bug where the client credentials were not being passed through the headers.
Requirements (place an
x
in each[ ]
)