feat: add users:read.email scope to default Slack app manifests#287
Open
blinkagent[bot] wants to merge 2 commits intomainfrom
Open
feat: add users:read.email scope to default Slack app manifests#287blinkagent[bot] wants to merge 2 commits intomainfrom
users:read.email scope to default Slack app manifests#287blinkagent[bot] wants to merge 2 commits intomainfrom
Conversation
Add the users:read.email OAuth scope to all default Slack app manifests and templates. This scope is required to access the email field in user profiles returned by the Slack Web API (users.info, users.list). Without it, the email field is omitted from responses even when users:read is present.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
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.
Summary
Add the
users:read.emailOAuth scope to all default Slack app manifests and templates.Why
The
users:read.emailscope is required to access theemailfield in user profiles returned by the Slack Web API (users.info,users.list). Without it, theemailfield is omitted from API responses even whenusers:readis present.From the Slack docs:
Changes
Added
users:read.emailto the bot scopes in:internal/site/lib/slack-manifest.tspackages/blink/src/cli/setup-slack-app.tspackages/slack/example-manifest.json(both bot and user scopes)packages/blink/src/cli/init-templates/scout/AGENTS.mdpackages/blink/src/cli/init-templates/scratch/AGENTS.mdpackages/blink/src/cli/init-templates/slack-bot/AGENTS.mdNote
Existing Slack app installations will need to be reinstalled for the new scope to take effect.
Created on behalf of @matifali