fix user name

src/store/channels-list/saga.test.ts

@@ -428,15 +428,17 @@ describe('channels list saga', () => {
   });
 
   describe(otherUserJoinedChannel, () => {
-    it('adds the user to the otherMember list', async () => {
+    it('finds the zOS user and adds the user to the otherMember list', async () => {
       const existingMembers = [
         { userId: 'user-1', matrixId: 'user-1' },
         { userId: 'user-2', matrixId: 'user-2' },
       ] as any;
-      const initialState = new StoreBuilder().withConversationList({
-        id: 'conversation-id',
-        otherMembers: existingMembers,
-      });
+      const initialState = new StoreBuilder()
+        .withConversationList({
+          id: 'conversation-id',
+          otherMembers: existingMembers,
+        })
+        .withUsers({ userId: 'new-user', matrixId: 'new-user', firstName: 'Jane', lastName: 'doe' });
 
       const { storeState } = await expectSaga(otherUserJoinedChannel, 'conversation-id', {
         userId: 'new-user',

src/store/channels-list/saga.ts

@@ -202,6 +202,10 @@ export function* createOptimisticConversation(userIds: string[], name: string =
 }
 
 export function* receiveCreatedConversation(conversation, optimisticConversation = { id: '', optimisticId: '' }) {
+  if (!conversation) {
+    return;
+  }
+
   const existingConversationsList = yield select(rawConversationsList());
   const listWithoutOptimistic = existingConversationsList.filter((id) => id !== optimisticConversation.id);
 
@@ -409,12 +413,17 @@ export function* otherUserJoinedChannel(roomId: string, user: User) {
     return;
   }
 
-  // TODO: Fetch user from zOS if we don't know about them yet
+  if (user.userId === user.matrixId) {
+    user = yield select(userByMatrixId, user.matrixId);
+  }
+
   if (!channel.otherMembers.includes(user.userId)) {
+    const otherMembers = [...channel.otherMembers, user];
     yield put(
       receiveChannel({
         id: channel.id,
-        otherMembers: [...channel.otherMembers, user],
+        isOneOnOne: otherMembers.length === 1,
+        otherMembers,
       })
     );
   }