-
Notifications
You must be signed in to change notification settings - Fork 126
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Bug Description
Slack adapter takes username as a fullname, but it's not intended to be public or full name as Slack UI says:
Usernames are not part of your profile, and are only required by Slack for technical reasons. Your username is mostly invisible to others, but you can change it if you want to.
Usernames must be all lowercase. They cannot be longer than 21 characters and can only contain letters, numbers, periods, hyphens, and underscores.
see https://<your_workspace>.slack.com/account/settings?updated_username=1#username
user.name is way better as a full name.
chat/packages/adapter-slack/src/index.ts
Line 1073 in af8c958
| userName: payload.user.username || payload.user.name || "unknown", |
Steps to Reproduce
- Integrate Slack adapter
- Send action block as a message
- Retrieve "full name" from
event.user.fullName
Expected Behavior
- Integrate Slack adapter
- Send action block as a message
- Retrieve "full name" from
event.user.fullName - It's something other than "username", such as "name"
Actual Behavior
- Integrate Slack adapter
- Send action block as a message
- Retrieve "full name" from
event.user.fullName - It's "username"
Code Sample
Chat SDK Version
4.20.2
Node.js Version
No response
Platform Adapter
Slack
Operating System
macOS
Additional Context
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working