small change

Author: bradwestfall

react/advanced-hooks/05-optimistic-ui/lecture/App.final.tsx

@@ -16,11 +16,10 @@ export function App() {
     setOptimisticLikes(optimisticLikes + 1)
     const data = (await updateDatabase(optimisticLikes + 1).then((r) => r.json())) as ResponseData
 
-    if (!data.error) {
-      console.log(data.likes)
-      setLikes(data.likes)
-    } else {
-      setLikes(data.likes)
+    setLikes(data.likes)
+    console.log(data.likes)
+
+    if (data.error) {
       setError(data.error)
     }
   }

react/advanced-hooks/05-optimistic-ui/lecture/NOTES.md

@@ -3,7 +3,7 @@
 Start with a basic form submission that asynchronously updates likes:
 
 1. Add our own implementation of `optimisticLikes` with `useState`. Notice how
-   `likes` becomes useless / unused. Set set it but nothing uses it. Also re-factor
+   `likes` becomes useless / unused. We set it but nothing uses it. Also re-factor
    to use transitions. See commented solution in App.tsx
 
 2. Refactor `onSubmit={submit}` to `action={action}` and now the homegrown optimistic likes

react/advanced-hooks/05-optimistic-ui/lecture/helpers/mockServer.ts

@@ -7,9 +7,9 @@ export async function updateDatabase(likes: number) {
   let status = 200
   let responseData: ResponseData = { likes }
 
-  if (likes === 5) {
+  if (likes >= 5) {
     status = 400
-    responseData = { likes: 4, error: 'Error saving 5 likes' }
+    responseData = { likes: 4, error: 'Error saving 5 or more likes' }
   }
 
   return new Response(JSON.stringify(responseData), {