Merge pull request #345 from GetStream/fix-edit-message-with-video-attachments

fix: Video attachments disappear during message edit #323

Author: szuperaz

projects/stream-chat-angular/src/lib/attachment.service.spec.ts

@@ -297,6 +297,7 @@ describe('AttachmentService', () => {
   it('should create attachmentUploads from attachments', () => {
     const imageFile = { name: 'flower.png' };
     const dataFile = { name: 'note.txt', size: 3272969 };
+    const videoFile = { name: 'cute.mov', size: 45367543 };
     const result = [
       {
         file: imageFile,
@@ -310,6 +311,12 @@ describe('AttachmentService', () => {
         url: 'url/to/data',
         type: 'file',
       },
+      {
+        file: videoFile,
+        state: 'success',
+        url: 'url/to/video',
+        type: 'video',
+      },
     ];
     const attachments = [
       { fallback: 'flower.png', image_url: 'url/to/img', type: 'image' },
@@ -319,6 +326,12 @@ describe('AttachmentService', () => {
         asset_url: 'url/to/data',
         type: 'file',
       },
+      {
+        title: 'cute.mov',
+        file_size: 45367543,
+        asset_url: 'url/to/video',
+        type: 'video',
+      },
     ];
     const spy = jasmine.createSpy();
     service.attachmentUploads$.subscribe(spy);

projects/stream-chat-angular/src/lib/attachment.service.ts

@@ -177,15 +177,15 @@ export class AttachmentService {
             name: attachment.fallback,
           } as File,
         });
-      } else if (attachment.type === 'file') {
+      } else if (attachment.type === 'file' || attachment.type === 'video') {
         attachmentUploads.push({
           url: attachment.asset_url,
           state: 'success',
           file: {
             name: attachment.title,
             size: attachment.file_size,
           } as File,
-          type: 'file',
+          type: attachment.type,
         });
       }
     });