From 1122d1daab0f7531d30a3b52e398242bf9daf877 Mon Sep 17 00:00:00 2001
From: sylenien <nikita@openreplay.com>
Date: Fri, 7 Oct 2022 17:34:05 +0200
Subject: [PATCH] change(ui): notes slack sharing

---
 .../Session_/Player/Controls/Time.js          |  1 -
 .../Player/Controls/components/CreateNote.tsx | 95 +++++++++++++------
 frontend/app/duck/sessions.js                 |  2 +-
 frontend/app/mstore/notesStore.ts             |  9 ++
 frontend/app/player/Player.ts                 |  4 +-
 frontend/app/player/store/store.js            |  6 +-
 frontend/app/services/NotesService.ts         | 11 +++
 7 files changed, 91 insertions(+), 37 deletions(-)

diff --git a/frontend/app/components/Session_/Player/Controls/Time.js b/frontend/app/components/Session_/Player/Controls/Time.js
index ca3c6ce4c..e8221b5b8 100644
--- a/frontend/app/components/Session_/Player/Controls/Time.js
+++ b/frontend/app/components/Session_/Player/Controls/Time.js
@@ -2,7 +2,6 @@ import React from 'react';
 import { Duration } from 'luxon';
 import { connectPlayer } from 'Player';
 import styles from './time.module.css';
-import { Tooltip } from 'react-tippy';
 
 const Time = ({ time, isCustom, format = 'm:ss', }) => (
   <div className={ !isCustom ? styles.time : undefined }>
diff --git a/frontend/app/components/Session_/Player/Controls/components/CreateNote.tsx b/frontend/app/components/Session_/Player/Controls/components/CreateNote.tsx
index 9856a4d69..ba2e11b8b 100644
--- a/frontend/app/components/Session_/Player/Controls/components/CreateNote.tsx
+++ b/frontend/app/components/Session_/Player/Controls/components/CreateNote.tsx
@@ -8,6 +8,8 @@ import stl from './styles.module.css';
 import { useStore } from 'App/mstore';
 import { toast } from 'react-toastify';
 import { injectNotes } from 'Player';
+import { fetchList as fetchSlack } from 'Duck/integrations/slack';
+import Select from 'Shared/Select';
 
 interface Props {
   isVisible: boolean;
@@ -18,6 +20,8 @@ interface Props {
   sessionId: string;
   isEdit: string;
   editNote: WriteNote;
+  slackChannels: Record<string, string>[];
+  fetchSlack: () => void;
 }
 
 function CreateNote({
@@ -29,12 +33,15 @@ function CreateNote({
   isEdit,
   editNote,
   updateNote,
+  slackChannels,
+  fetchSlack,
 }: Props) {
   const [text, setText] = React.useState('');
+  const [channel, setChannel] = React.useState('');
   const [isPublic, setPublic] = React.useState(false);
   const [tag, setTag] = React.useState<iTag>(TAGS[0]);
   const [useTimestamp, setUseTs] = React.useState(true);
-  const inputRef = React.createRef<HTMLTextAreaElement>()
+  const inputRef = React.createRef<HTMLTextAreaElement>();
   const { notesStore } = useStore();
 
   React.useEffect(() => {
@@ -50,14 +57,12 @@ function CreateNote({
 
   React.useEffect(() => {
     if (inputRef.current) {
-      inputRef.current.focus()
+      fetchSlack();
+      inputRef.current.focus();
     }
-  }, [isVisible])
+  }, [isVisible]);
 
   const duration = Duration.fromMillis(time).toFormat('mm:ss');
-  const stopEvents = (e: any) => {
-    e.stopPropagation();
-  };
 
   const onSubmit = () => {
     if (text === '') return;
@@ -68,29 +73,37 @@ function CreateNote({
       timestamp: useTimestamp ? (isEdit ? editNote.timestamp : time) : -1,
       isPublic,
     };
-
+    const onSuccess = (noteId: string) => {
+      if (channel) {
+        notesStore.sendSlackNotification(noteId, channel)
+      }
+    }
     if (isEdit) {
-      return notesStore.updateNote(editNote.noteId, note).then((r) => {
-        toast.success('Note updated');
-        notesStore.fetchSessionNotes(sessionId).then((notes) => {
-          injectNotes(notes);
-          updateNote(r);
+      return notesStore
+        .updateNote(editNote.noteId, note)
+        .then((r) => {
+          toast.success('Note updated');
+          notesStore.fetchSessionNotes(sessionId).then((notes) => {
+            injectNotes(notes);
+            onSuccess(editNote.noteId)
+            updateNote(r);
+          });
+        })
+        .catch((e) => {
+          toast.error('Error updating note');
+          console.error(e);
+        })
+        .finally(() => {
+          setCreateNoteTooltip({ isVisible: false, time: 0 });
+          setText('');
+          setTag(undefined);
         });
-      })
-      .catch((e) => {
-        toast.error('Error updating note');
-        console.error(e);
-      })
-      .finally(() => {
-        setCreateNoteTooltip({ isVisible: false, time: 0 });
-        setText('');
-        setTag(undefined);
-      });
     }
 
     return notesStore
       .addNote(sessionId, note)
       .then((r) => {
+        onSuccess(r.noteId as unknown as string)
         toast.success('Note added');
         notesStore.fetchSessionNotes(sessionId).then((notes) => {
           injectNotes(notes);
@@ -109,7 +122,7 @@ function CreateNote({
   };
 
   const closeTooltip = () => {
-    setCreateNoteTooltip({ isVisible: false, time: 0 });
+    setCreateNoteTooltip({ isVisible: false, time: 100 });
   };
 
   const tagActive = (noteTag: iTag) => tag === noteTag;
@@ -118,17 +131,27 @@ function CreateNote({
     setTag(tag);
   };
 
+  const slackChannelsOptions = slackChannels.map(({ webhookId, name }) => ({
+    value: webhookId,
+    label: name,
+  }));
+
+  const changeChannel = ({ value, name }: { value: string; name: string }) => {
+    setChannel(value);
+  };
+
   return (
     <div
       className={stl.noteTooltip}
       style={{
-        top: -260,
+        top: -320,
         width: 350,
         left: 'calc(50% - 175px)',
         display: isVisible ? 'flex' : 'none',
         flexDirection: 'column',
         gap: '1rem',
       }}
+      onClick={(e) => e.stopPropagation()}
     >
       <div className="flex items-center bg-gray-lightest">
         <Icon name="quotes" size={20} />
@@ -176,13 +199,23 @@ function CreateNote({
             onClick={() => addTag(tag)}
           >
             {tagActive(tag) ? <Icon name="check-circle-fill" color="white" size={13} /> : null}
-            <div>
-              {tag}
-            </div>
+            <div>{tag}</div>
           </div>
         ))}
       </div>
 
+      {slackChannelsOptions.length > 0 ? (
+        <div>
+          <Select
+            options={slackChannelsOptions}
+            defaultValue={"Share to Slack"}
+            // @ts-ignore
+            onChange={changeChannel}
+            className="mr-4"
+          />
+        </div>
+      ) : null}
+
       <div className="flex">
         <Button variant="primary" className="mr-4" disabled={text === ''} onClick={onSubmit}>
           Add Note
@@ -204,11 +237,13 @@ export default connect(
       time = 0,
       isEdit,
       note: editNote,
-    // @ts-ignore
+      // @ts-ignore
     } = state.getIn(['sessions', 'createNoteTooltip']);
     // @ts-ignore
+    const slackChannels = state.getIn(['slack', 'list']);
+    // @ts-ignore
     const sessionId = state.getIn(['sessions', 'current', 'sessionId']);
-    return { isVisible, time, sessionId, isEdit, editNote };
+    return { isVisible, time, sessionId, isEdit, editNote, slackChannels };
   },
-  { setCreateNoteTooltip, addNote, updateNote }
+  { setCreateNoteTooltip, addNote, updateNote, fetchSlack }
 )(CreateNote);
diff --git a/frontend/app/duck/sessions.js b/frontend/app/duck/sessions.js
index f959619ea..a76a38c94 100644
--- a/frontend/app/duck/sessions.js
+++ b/frontend/app/duck/sessions.js
@@ -28,7 +28,7 @@ const SET_TIMELINE_POINTER = 'sessions/SET_TIMELINE_POINTER';
 const SET_TIMELINE_HOVER_POINTER = 'sessions/SET_TIMELINE_HOVER_POINTER';
 
 const SET_CREATE_NOTE_TOOLTIP = 'sessions/SET_CREATE_NOTE_TOOLTIP'
-const SET_EDIT_NOTE_TOOLTIP = 'sessions/SET_CREATE_NOTE_TOOLTIP'
+const SET_EDIT_NOTE_TOOLTIP = 'sessions/SET_EDIT_NOTE_TOOLTIP'
 const FILTER_OUT_NOTE = 'sessions/FILTER_OUT_NOTE'
 const ADD_NOTE = 'sessions/ADD_NOTE'
 const UPDATE_NOTE = 'sessions/UPDATE_NOTE'
diff --git a/frontend/app/mstore/notesStore.ts b/frontend/app/mstore/notesStore.ts
index a5b83f879..9715c3e87 100644
--- a/frontend/app/mstore/notesStore.ts
+++ b/frontend/app/mstore/notesStore.ts
@@ -125,4 +125,13 @@ export default class NotesStore {
 
     this.fetchNotes()
   }
+
+  async sendSlackNotification(noteId: string, webhook: string) {
+    try {
+      const resp = await notesService.sendSlackNotification(noteId, webhook)
+      return resp
+    } catch (e) {
+      console.error(e)
+    }
+  }
 }
diff --git a/frontend/app/player/Player.ts b/frontend/app/player/Player.ts
index b37f44811..181350cbe 100644
--- a/frontend/app/player/Player.ts
+++ b/frontend/app/player/Player.ts
@@ -158,10 +158,10 @@ export default class Player extends MessageDistributor {
     }
   }
 
-  jump(time = getState().time, index: number) {
+  jump(setTime: number, index: number) {
     const { live, liveTimeTravel, endTime } = getState();
     if (live && !liveTimeTravel) return;
-
+    const time = setTime ? setTime : getState().time
     if (getState().playing) {
       cancelAnimationFrame(this._animationFrameRequestId);
       // this._animationFrameRequestId = requestAnimationFrame(() => {
diff --git a/frontend/app/player/store/store.js b/frontend/app/player/store/store.js
index a09ba881a..e8faf9909 100644
--- a/frontend/app/player/store/store.js
+++ b/frontend/app/player/store/store.js
@@ -1,7 +1,7 @@
 import { createStore } from 'redux';
-import reducer, { 
-	update as updateAction, 
-	clean as cleanAction, 
+import reducer, {
+	update as updateAction,
+	clean as cleanAction,
 } from './duck';
 
 const store = createStore(reducer);
diff --git a/frontend/app/services/NotesService.ts b/frontend/app/services/NotesService.ts
index 441a1a43e..d99b40013 100644
--- a/frontend/app/services/NotesService.ts
+++ b/frontend/app/services/NotesService.ts
@@ -108,4 +108,15 @@ export default class NotesService {
         }
       })
     }
+
+    sendSlackNotification(noteId: number, webhook: string) {
+      return this.client.get(`/notes/${noteId}/slack/${webhook}`)
+      .then(r => {
+        if (r.ok) {
+          return r.json().then(r => r.data)
+        } else {
+          throw new Error('Error sending slack notif: ' + r.status)
+        }
+      })
+    }
 }