previously the RelationshipPrompt component was getting rendered in sidebar chats on web. This diff adds the proper check to prevent this behavior.
Details
Details
Before:
After:
Test ensuring that relationship prompt still shows on Personal threads
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- master
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/chat/chat-message-list.react.js | ||
---|---|---|
193 | here i only initialized to null to achieve consistency with native, if that's at all important |
Comment Actions
Regarding the test plan, you also have to check if the prompt is still displayed in PERSONAL threads.
web/chat/chat-message-list.react.js | ||
---|---|---|
193 | It's ok both ways |