Fix #799 Add OpenID Connect document#805
Merged
Merged
Conversation
seratch
commented
Aug 10, 2021
| ### Slack Config for Distributing Your Slack App | ||
|
|
||
| Here is the list of the necessary configurations for distributing apps built with Bolt. If you prefer using other env variable names or other solutions to load this information, implement your own way to load **SlackConfig** instead. | ||
| Here is the list of the necessary configurations for distributing apps built with Bolt. If you prefer using other env variable names or other solutions to load this information, implement your own way to load **AppConfig** instead. |
Contributor
Author
There was a problem hiding this comment.
This has been wrong from the beginning 🤦
|
|
||
| # Sign in with Slack (OpenID Connect) | ||
|
|
||
| [Sign in with Slack](https://api.slack.com/authentication/sign-in-with-slack) helps users log into your service using their Slack profile. The platform feature was recently upgraded to be compatible with the standard [OpenID Connect](https://openid.net/connect/) specification. With Bolt for Java v1.10 or higher, implementing the auth flow is much easier. |
Contributor
Author
There was a problem hiding this comment.
will merge this doc update after releasing v1.10.0
Codecov Report
@@ Coverage Diff @@
## main #805 +/- ##
============================================
+ Coverage 76.75% 76.80% +0.05%
- Complexity 3395 3397 +2
============================================
Files 380 380
Lines 10224 10224
Branches 997 997
============================================
+ Hits 7847 7853 +6
+ Misses 1800 1796 -4
+ Partials 577 575 -2
Continue to review full report at Codecov.
|
misscoded
approved these changes
Aug 10, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request resolves #799 by adding the OpenID Connect document page.
The Japanese version will come up later.DONECategory (place an
xin each of the[ ])Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to the those rules.