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.
|
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
Bug Description
Slack adapter takes username as a fullname, but it's not intended to be public or full name as Slack UI says:
user.nameis way better as a full name.chat/packages/adapter-slack/src/index.ts
Line 1073 in af8c958
Steps to Reproduce
event.user.fullNameExpected Behavior
event.user.fullNameActual Behavior
event.user.fullNameCode Sample
Chat SDK Version
4.20.2
Node.js Version
No response
Platform Adapter
Slack
Operating System
macOS
Additional Context
No response