From 2f222aebded186d2782808a0f797dc70f895f23d Mon Sep 17 00:00:00 2001
From: Anoop M D <anoop.md1421@gmail.com>
Date: Tue, 18 Jan 2022 21:41:27 +0530
Subject: [PATCH] feat: moved the logic up for managing request url

---
 .../src/components/QueryUrl/index.js          |  3 +-
 .../src/components/RequestTabPanel/index.js   | 29 ++++++++++++-------
 .../src/providers/Store/actions.js            |  2 ++
 .../src/providers/Store/reducer.js            | 15 ++++++++++
 4 files changed, 37 insertions(+), 12 deletions(-)

diff --git a/packages/grafnode-components/src/components/QueryUrl/index.js b/packages/grafnode-components/src/components/QueryUrl/index.js
index 95f6fbb21..4141f5874 100644
--- a/packages/grafnode-components/src/components/QueryUrl/index.js
+++ b/packages/grafnode-components/src/components/QueryUrl/index.js
@@ -85,7 +85,8 @@ const QueryUrl = ({value, onChange, handleRun, collections}) => {
       <div className="flex items-center flex-grow input-container h-full">
         <input
           className="px-3 w-full"
-          type="text" value={value} onChange={(event) => onChange(event.target.value)}
+          type="text" defaultValue={value}
+          onChange={(event) => onChange(event.target.value)}
         />
       </div>
       <button
diff --git a/packages/grafnode-components/src/components/RequestTabPanel/index.js b/packages/grafnode-components/src/components/RequestTabPanel/index.js
index abdc576d2..69d892518 100644
--- a/packages/grafnode-components/src/components/RequestTabPanel/index.js
+++ b/packages/grafnode-components/src/components/RequestTabPanel/index.js
@@ -18,7 +18,6 @@ const RequestTabPanel = ({dispatch, actions, collections, activeRequestTabId, re
   }
 
   let asideWidth = 200;
-  let [url, setUrl] = useState('https://api.spacex.land/graphql');
   let {
     schema 
   } = useGraphqlSchema('https://api.spacex.land/graphql');
@@ -41,17 +40,25 @@ const RequestTabPanel = ({dispatch, actions, collections, activeRequestTabId, re
     e.preventDefault();
     setDragging(true);
   };
-  useEffect(() => {
-    document.addEventListener('mouseup', handleMouseUp);
-    document.addEventListener('mousemove', handleMouseMove);
+  // useEffect(() => {
+  //   document.addEventListener('mouseup', handleMouseUp);
+  //   document.addEventListener('mousemove', handleMouseMove);
 
-    return () => {
-      document.removeEventListener('mouseup', handleMouseUp);
-      document.removeEventListener('mousemove', handleMouseMove);
-    };
-  }, [dragging, leftPaneWidth]);
+  //   return () => {
+  //     document.removeEventListener('mouseup', handleMouseUp);
+  //     document.removeEventListener('mousemove', handleMouseMove);
+  //   };
+  // }, [dragging, leftPaneWidth]);
+
+  const onUrlChange = (value) => {
+    dispatch({
+      type: actions.REQUEST_URL_CHANGED,
+      url: value,
+      requestTab: focusedTab,
+      collectionId: collection ? collection.id : null
+    });
+  };
 
-  const onUrlChange = (value) => setUrl(value);
   const onQueryChange = (value) => setQuery(value);
 
   if(!activeRequestTabId) {
@@ -97,7 +104,7 @@ const RequestTabPanel = ({dispatch, actions, collections, activeRequestTabId, re
       >
         <div className="pt-2 text-gray-600">{item.name}</div>
         <QueryUrl
-          value = {url}
+          value = {item.request.url}
           onChange={onUrlChange}
           handleRun={runQuery}
           collections={collections}
diff --git a/packages/grafnode-run/src/providers/Store/actions.js b/packages/grafnode-run/src/providers/Store/actions.js
index 0d91c3f2f..390e74c27 100644
--- a/packages/grafnode-run/src/providers/Store/actions.js
+++ b/packages/grafnode-run/src/providers/Store/actions.js
@@ -2,6 +2,7 @@ const SIDEBAR_COLLECTION_CLICK = "SIDEBAR_COLLECTION_CLICK";
 const SIDEBAR_COLLECTION_ITEM_CLICK = "SIDEBAR_COLLECTION_ITEM_CLICK";
 const REQUEST_TAB_CLICK = "REQUEST_TAB_CLICK";
 const REQUEST_TAB_CLOSE = "REQUEST_TAB_CLOSE";
+const REQUEST_URL_CHANGED = "REQUEST_URL_CHANGED";
 const RESPONSE_RECEIVED = "RESPONSE_RECEIVED";
 const SEND_REQUEST = "SEND_REQUEST";
 const SENDING_REQUEST = "SENDING_REQUEST";
@@ -14,6 +15,7 @@ export default {
   SIDEBAR_COLLECTION_ITEM_CLICK,
   REQUEST_TAB_CLICK,
   REQUEST_TAB_CLOSE,
+  REQUEST_URL_CHANGED,
   RESPONSE_RECEIVED,
   SEND_REQUEST,
   SENDING_REQUEST,
diff --git a/packages/grafnode-run/src/providers/Store/reducer.js b/packages/grafnode-run/src/providers/Store/reducer.js
index b5b1ffffb..8ef05b3e5 100644
--- a/packages/grafnode-run/src/providers/Store/reducer.js
+++ b/packages/grafnode-run/src/providers/Store/reducer.js
@@ -58,6 +58,21 @@ const reducer = (state, action) => {
       });
     }
 
+    case actions.REQUEST_URL_CHANGED: {
+      return produce(state, (draft) => {
+        const collection = find(draft.collections, (c) => c.id === action.collectionId);
+
+        if(collection) {
+          let flattenedItems = flattenItems(collection.items);
+          let item = findItem(flattenedItems, action.requestTab.id);
+          
+          if(item) {
+            item.request.url = action.url;
+          }
+        }
+      });
+    }
+
     case actions.ADD_NEW_HTTP_REQUEST: {
       return produce(state, (draft) => {
         const uid = nanoid();