fix rating update issue, relocate recipe state to App and persist ran…

…dom recipe after like
mykovasyl · Mar 29, 2023 · 0c67d3f · 0c67d3f
1 parent 44b3b43
commit 0c67d3f
Show file tree

Hide file tree

Showing 4 changed files with 16 additions and 16 deletions.
diff --git a/app/controllers/recipes_controller.rb b/app/controllers/recipes_controller.rb
@@ -10,7 +10,7 @@ def create
   def update
     recipe_to_update = find_recipe
     recipe_to_update.update!(update_recipe_params) 
-    render json: recipe, status: :accepted
+    render json: recipe_to_update, status: :accepted
   end
 
   def destroy

diff --git a/client/src/components/App.js b/client/src/components/App.js
@@ -26,6 +26,16 @@ export const UserContext = createContext();
 function App() {
   const [currentUser, setCurrentUser] = useState({});
   const [errors, setErrors] = useState([]);
+  const [recipe, setRecipe] = useState({
+    title: "",
+    readyIn: "",
+    image: "",
+    summary: "",
+    instructions: "",
+    ingredients: [],
+    sourceURL: "",
+    user_id: null,
+  });
 
   const navigate = useNavigate();
 
@@ -152,6 +162,8 @@ function App() {
     <UserContext.Provider
       value={{
         currentUser,
+        recipe,
+        setRecipe,
         setCurrentUser,
         handleLogOut,
         handleAddRecipe,

diff --git a/client/src/components/RandomRecipe.js b/client/src/components/RandomRecipe.js
@@ -4,18 +4,7 @@ import { UserContext } from "./App";
 import Recipe from "./Recipe";
 
 function RandomRecipe() {
-  const [recipe, setRecipe] = useState({
-    title: "",
-    readyIn: "",
-    image: "",
-    summary: "",
-    instructions: "",
-    ingredients: [],
-    sourceURL: "",
-    user_id: null,
-  });
-  const { handleAddRecipe } = useContext(UserContext);
-  const [isLoaded, setIsLoaded] = useState(false);
+  const { handleAddRecipe, recipe, setRecipe } = useContext(UserContext);
 
   function handleClick() {
     fetch(
@@ -38,7 +27,6 @@ function RandomRecipe() {
           sourceURL: data.recipes[0].spoonacularSourceUrl,
           user_id: null,
         });
-        setIsLoaded(true);
       });
   }
 
@@ -47,7 +35,7 @@ function RandomRecipe() {
       <Button type='button' variant='contained' onClick={handleClick}>
         Get Random Recipe
       </Button>
-      {isLoaded ? (
+      {recipe.title ? (
         <Recipe recipe={recipe} handleAddRecipe={handleAddRecipe} />
       ) : null}
     </div>

diff --git a/client/src/components/Recipe.js b/client/src/components/Recipe.js
@@ -81,7 +81,7 @@ function Recipe({ recipe }) {
   return (
     <Card sx={{ maxWidth: 500 }}>
       <CardHeader title={title} subheader={readyInText} />
-      {user_id ? (
+      {user_id && recipe.id ? (
         <Rating
           name='simple-controlled'
           value={ratingValue}