got rid oof diffs for workspace builds

Kira-Pilot · Kira-Pilot · commit e17abf137668 · 2022-10-20T15:43:37.000Z
diff --git a/enterprise/audit/table.go b/enterprise/audit/table.go
@@ -101,19 +101,20 @@ var AuditableResources = auditMap(map[any]map[string]Action{
 		"ttl":                ActionTrack,
 		"last_used_at":       ActionIgnore,
 	},
+	// We don't show any diff for the WorkspaceBuild resource
 	&database.WorkspaceBuild{}: {
-		"id":                  ActionIgnore, // Unimportant to the user
-		"created_at":          ActionIgnore, // Never changes.
-		"updated_at":          ActionIgnore, // Changes, but is implicit and not helpful in a diff.
-		"workspace_id":        ActionTrack,
-		"template_version_id": ActionTrack,
-		"build_number":        ActionIgnore, // Unimportant to the user
-		"transition":          ActionTrack,
-		"initiator_id":        ActionIgnore, // Changes, but is implicit and not helpful in a diff.
-		"provisioner_state":   ActionIgnore, // Unimportant to the user
-		"job_id":              ActionIgnore, // Unimportant to the user
-		"deadline":            ActionIgnore, // Unimportant to the user
-		"reason":              ActionIgnore, // Unimportant to the user
+		"id":                  ActionIgnore,
+		"created_at":          ActionIgnore,
+		"updated_at":          ActionIgnore,
+		"workspace_id":        ActionIgnore,
+		"template_version_id": ActionIgnore,
+		"build_number":        ActionIgnore,
+		"transition":          ActionIgnore,
+		"initiator_id":        ActionIgnore,
+		"provisioner_state":   ActionIgnore,
+		"job_id":              ActionIgnore,
+		"deadline":            ActionIgnore,
+		"reason":              ActionIgnore,
 	},
 })
 
diff --git a/site/src/components/AuditLogRow/AuditLogRow.tsx b/site/src/components/AuditLogRow/AuditLogRow.tsx
@@ -130,13 +130,11 @@ export const AuditLogRow: React.FC<AuditLogRowProps> = ({
             </Stack>
           </Stack>
 
-          <div
-            className={
-              shouldDisplayDiff ? undefined : styles.disabledDropdownIcon
-            }
-          >
-            {isDiffOpen ? <CloseDropdown /> : <OpenDropdown />}
-          </div>
+          {shouldDisplayDiff ? (
+            <div> {isDiffOpen ? <CloseDropdown /> : <OpenDropdown />}</div>
+          ) : (
+            <div className={styles.columnWithoutDiff}></div>
+          )}
         </Stack>
 
         {shouldDisplayDiff && (
@@ -190,8 +188,8 @@ const useStyles = makeStyles((theme) => ({
     color: theme.palette.text.secondary,
     whiteSpace: "nowrap",
   },
-
-  disabledDropdownIcon: {
-    opacity: 0.5,
+  // offset the absence of the arrow icon on diff-less logs
+  columnWithoutDiff: {
+    marginLeft: "24px",
   },
 }))
