Address PR review

BrunoQuaresma · BrunoQuaresma · commit 6cf89a44d68c · 2022-08-31T19:04:03.000Z
diff --git a/site/src/components/AuditLogRow/AuditLogDiff.tsx b/site/src/components/AuditLogRow/AuditLogDiff.tsx
@@ -11,7 +11,7 @@ const getDiffValue = (value: number | string | boolean) => {
   return value.toString()
 }
 
-export const AuditDiff: React.FC<{ diff: AuditLog["diff"] }> = ({ diff }) => {
+export const AuditLogDiff: React.FC<{ diff: AuditLog["diff"] }> = ({ diff }) => {
   const styles = useStyles()
   const diffEntries = Object.entries(diff)
 
diff --git a/site/src/components/AuditLogRow/AuditLogRow.tsx b/site/src/components/AuditLogRow/AuditLogRow.tsx
@@ -11,14 +11,14 @@ import { UserAvatar } from "components/UserAvatar/UserAvatar"
 import { ComponentProps, useState } from "react"
 import { Link as RouterLink } from "react-router-dom"
 import { createDayString } from "util/createDayString"
-import { AuditDiff } from "./AuditLogDiff"
+import { AuditLogDiff } from "./AuditLogDiff"
 
 const pillTypeByHttpStatus = (httpStatus: number): ComponentProps<typeof Pill>["type"] => {
   if (httpStatus >= 300 && httpStatus < 500) {
     return "warning"
   }
 
-  if (httpStatus > 500) {
+  if (httpStatus >= 500) {
     return "error"
   }
 
@@ -171,7 +171,7 @@ export const AuditLogRow: React.FC<AuditLogRowProps> = ({
 
         {shouldDisplayDiff && (
           <Collapse in={isDiffOpen}>
-            <AuditDiff diff={auditLog.diff} />
+            <AuditLogDiff diff={auditLog.diff} />
           </Collapse>
         )}
       </TableCell>

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ const getDiffValue = (value: number \| string \| boolean) => {`
`11`	`11`	`return value.toString()`
`12`	`12`	`}`
`13`	`13`
`14`		`-export const AuditDiff: React.FC<{ diff: AuditLog["diff"] }> = ({ diff }) => {`
	`14`	`+export const AuditLogDiff: React.FC<{ diff: AuditLog["diff"] }> = ({ diff }) => {`
`15`	`15`	`const styles = useStyles()`
`16`	`16`	`const diffEntries = Object.entries(diff)`
`17`	`17`