Fix attachment duplication on add

client/src/sagas/app/requests/attachment.js

@@ -23,7 +23,7 @@ export function* createAttachmentRequest(cardId, localId, data) {
   );
 
   try {
-    const { item } = yield call(request, api.createAttachment, cardId, data);
+    const { item } = yield call(request, api.createAttachment, cardId, data, localId);
 
     const action = createAttachmentSucceeded(localId, item);
     yield put(action);

client/src/sagas/app/services/socket.js

@@ -5,6 +5,7 @@ import { logoutService } from './login';
 import { closeModalService } from './modal';
 import { deleteNotificationsRequest, fetchUsersRequest } from '../requests';
 import {
+  attachmentWithIdExistsSelector,
   currentModalSelector,
   currentUserIdSelector,
   currentUserSelector,
@@ -208,8 +209,12 @@ export function* deleteTaskReceivedService(task) {
   yield put(deleteTaskReceived(task));
 }
 
-export function* createAttachmentReceivedService(attachment) {
-  yield put(createAttachmentReceived(attachment));
+export function* createAttachmentReceivedService(attachment, requestId) {
+  const exists = yield select(attachmentWithIdExistsSelector, requestId);
+
+  if (!exists) {
+    yield put(createAttachmentReceived(attachment));
+  }
 }
 
 export function* updateAttachmentReceivedService(attachment) {

client/src/sagas/app/watchers/socket.js

@@ -156,8 +156,8 @@ const createSocketEventsChannel = () =>
       emit([deleteTaskReceivedService, item]);
     };
 
-    const handleAttachmentCreate = api.makeHandleAttachmentCreate(({ item }) => {
-      emit([createAttachmentReceivedService, item]);
+    const handleAttachmentCreate = api.makeHandleAttachmentCreate(({ item, requestId }) => {
+      emit([createAttachmentReceivedService, item, requestId]);
     });
 
     const handleAttachmentUpdate = api.makeHandleAttachmentUpdate(({ item }) => {