From 5d82a39af38616e8443679b3ea0ddec809f2b833 Mon Sep 17 00:00:00 2001
From: mike12345567 <me@michaeldrury.co.uk>
Date: Fri, 17 Jun 2022 17:00:43 +0100
Subject: [PATCH] Adding edit automation button.

---
 .../portal/overview/HistoryDetailsPanel.svelte       | 12 +++++++++---
 .../src/components/portal/overview/HistoryTab.svelte |  3 ++-
 .../portal/overview/[application]/index.svelte       |  2 +-
 packages/server/src/automations/logging/index.ts     |  3 +--
 4 files changed, 13 insertions(+), 7 deletions(-)

diff --git a/packages/builder/src/components/portal/overview/HistoryDetailsPanel.svelte b/packages/builder/src/components/portal/overview/HistoryDetailsPanel.svelte
index bc62f38502..cb384c8007 100644
--- a/packages/builder/src/components/portal/overview/HistoryDetailsPanel.svelte
+++ b/packages/builder/src/components/portal/overview/HistoryDetailsPanel.svelte
@@ -3,8 +3,10 @@
   import StatusRenderer from "components/portal/overview/StatusRenderer.svelte"
   import DateTimeRenderer from "components/common/renderers/DateTimeRenderer.svelte"
   import TestDisplay from "components/automation/AutomationBuilder/TestDisplay.svelte"
+  import { goto } from "@roxi/routify"
 
   export let history
+  export let appId
   export let close
 </script>
 
@@ -23,10 +25,14 @@
       </div>
       <div class="icon">
         <Icon name="JourneyVoyager" />
-        <div>{history.name}</div>
+        <div>{history.automationName}</div>
       </div>
       <div>
-        <ActionButton icon="Edit" fullWidth={false}
+        <ActionButton
+          icon="Edit"
+          fullWidth={false}
+          on:click={() =>
+            $goto(`../../../app/${appId}/automate/${history.automationId}`)}
           >Edit automation</ActionButton
         >
       </div>
@@ -55,7 +61,7 @@
   .bottom {
     margin-top: var(--spacing-m);
     border-top: var(--border-light);
-    padding-top: var(--spacing-xl);
+    padding-top: calc(var(--spacing-xl) * 2);
     height: 100%;
   }
 
diff --git a/packages/builder/src/components/portal/overview/HistoryTab.svelte b/packages/builder/src/components/portal/overview/HistoryTab.svelte
index 0feeb7c7e1..a2b9879d54 100644
--- a/packages/builder/src/components/portal/overview/HistoryTab.svelte
+++ b/packages/builder/src/components/portal/overview/HistoryTab.svelte
@@ -6,7 +6,7 @@
   import { automationStore } from "builderStore"
   import { onMount } from "svelte"
 
-  export let appId
+  export let app
 
   let runHistory = []
   let showPanel = false
@@ -140,6 +140,7 @@
   </Layout>
   <div class="panel" class:panelShow={showPanel}>
     <HistoryDetailsPanel
+      appId={app.devId}
       bind:history={selectedHistory}
       close={() => {
         showPanel = false
diff --git a/packages/builder/src/pages/builder/portal/overview/[application]/index.svelte b/packages/builder/src/pages/builder/portal/overview/[application]/index.svelte
index e470167309..38988af0dd 100644
--- a/packages/builder/src/pages/builder/portal/overview/[application]/index.svelte
+++ b/packages/builder/src/pages/builder/portal/overview/[application]/index.svelte
@@ -301,7 +301,7 @@
             />
           </Tab>
           <Tab title="Automation History">
-            <HistoryTab appId={selectedApp?._id} />
+            <HistoryTab app={selectedApp} />
           </Tab>
           {#if false}
             <Tab title="Backups">
diff --git a/packages/server/src/automations/logging/index.ts b/packages/server/src/automations/logging/index.ts
index dfb3ef81ed..b340709061 100644
--- a/packages/server/src/automations/logging/index.ts
+++ b/packages/server/src/automations/logging/index.ts
@@ -1,5 +1,4 @@
 import {
-  AutomationLog,
   AutomationLogPage,
   AutomationResults,
   AutomationStatus,
@@ -135,7 +134,7 @@ export async function storeLog(
   automation: Automation,
   results: AutomationResults
 ) {
-  // can disable this if un-needed in self host
+  // can disable this if un-needed in self-host
   if (env.DISABLE_AUTOMATION_LOGS) {
     return
   }