When a user can't be added to thick thread, they shouldn't be included in search results.
Depends on D13404
Details
Details
Checked that the user who don't support thick threads aren't shown in add member modals of thick threads.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- improvements2
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| lib/shared/search-utils.js | ||
|---|---|---|
| 166 | In what cases is threadType not set here? | |
| lib/shared/search-utils.js | ||
|---|---|---|
| 166 | This happens in chat-thread-composer.react.js and message-list-container.react.js where we don't know what the resulting thread type would be. In these cases, we shouldn't filter out users who don't have CSAT because we're determining the thread thickness later, based on the presence of the CSAT. | |
| lib/shared/search-utils.js | ||
|---|---|---|
| 166 | Thanks for explaining! | |