refactor: replace "todos" with todoItem in useTodoAPI

Author: sebastianvitterso

web/src/features/todos/todo-list/TodoItem.tsx

@@ -7,16 +7,16 @@ import { useTodoAPI } from '../../../hooks/useTodoAPI'
 import { StyledTodoItemTitle } from './TodoItem.styled'
 
 const TodoItem = ({ todo }: { todo: AddTodoResponse }) => {
-  const { toggleTodo, removeTodo } = useTodoAPI()
+  const { toggleTodoItem, removeTodoItem } = useTodoAPI()
   const { dispatch } = useTodos()
 
   async function toggle() {
-    await toggleTodo(todo)
+    await toggleTodoItem(todo)
     dispatch({ type: 'TOGGLE_TODO', payload: todo })
   }
 
   async function remove() {
-    await removeTodo(todo)
+    await removeTodoItem(todo)
     dispatch({ type: 'REMOVE_TODO', payload: todo })
   }
 

web/src/features/todos/todo-list/TodoList.tsx

@@ -7,14 +7,14 @@ import TodoItem from './TodoItem'
 import { StyledInput, StyledTodoList } from './TodoList.styled'
 
 const AddItem = () => {
-  const { addTodo } = useTodoAPI()
+  const { addTodoItem } = useTodoAPI()
   const { dispatch } = useTodos()
   const [value, setValue] = useState('')
 
   const add: FormEventHandler = (event) => {
     event.preventDefault()
     if (value) {
-      addTodo(value).then((todo) => dispatch({ type: 'ADD_TODO', payload: todo }))
+      addTodoItem(value).then((todo) => dispatch({ type: 'ADD_TODO', payload: todo }))
     }
     setValue('')
   }
@@ -39,12 +39,12 @@ const AddItem = () => {
 }
 
 const TodoList = () => {
-  const { getAllTodos } = useTodoAPI()
+  const { getAllTodoItems } = useTodoAPI()
   const { state, dispatch } = useTodos()
 
   useEffect(() => {
-    getAllTodos().then((todos) => dispatch({ type: 'INITIALIZE', payload: todos }))
-  }, [dispatch, getAllTodos])
+    getAllTodoItems().then((todos) => dispatch({ type: 'INITIALIZE', payload: todos }))
+  }, [dispatch, getAllTodoItems])
 
   return (
     <StyledTodoList>

web/src/hooks/useTodoAPI.tsx

@@ -2,7 +2,7 @@ import { useCallback } from 'react'
 import { type AddTodoResponse, ApiError, type ErrorResponse, TodosService } from '../api/generated'
 
 export function useTodoAPI() {
-  const addTodo = useCallback(async (title: string) => {
+  const addTodoItem = useCallback(async (title: string) => {
     return TodosService.create({ title }).catch((error) => {
       if (error instanceof ApiError) {
         console.error((error.body as ErrorResponse).message)
@@ -11,7 +11,7 @@ export function useTodoAPI() {
     })
   }, [])
 
-  const getAllTodos = useCallback(async () => {
+  const getAllTodoItems = useCallback(async () => {
     return TodosService.getAll().catch((error) => {
       if (error instanceof ApiError) {
         console.error((error.body as ErrorResponse).message)
@@ -20,7 +20,7 @@ export function useTodoAPI() {
     })
   }, [])
 
-  const toggleTodo = useCallback(async (todo: AddTodoResponse) => {
+  const toggleTodoItem = useCallback(async (todo: AddTodoResponse) => {
     return TodosService.updateById(todo.id, {
       is_completed: !todo.is_completed,
       title: todo.title,
@@ -32,7 +32,7 @@ export function useTodoAPI() {
     })
   }, [])
 
-  const removeTodo = useCallback(async (todo: AddTodoResponse) => {
+  const removeTodoItem = useCallback(async (todo: AddTodoResponse) => {
     return TodosService.deleteById(todo.id).catch((error) => {
       if (error instanceof ApiError) {
         console.error((error.body as ErrorResponse).message)
@@ -41,5 +41,5 @@ export function useTodoAPI() {
     })
   }, [])
 
-  return { addTodo, getAllTodos, toggleTodo, removeTodo }
+  return { addTodoItem, getAllTodoItems, toggleTodoItem, removeTodoItem }
 }