Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unnecessary Code #15

Open
pabloCabrera7 opened this issue Jun 19, 2020 · 1 comment
Open

Unnecessary Code #15

pabloCabrera7 opened this issue Jun 19, 2020 · 1 comment

Comments

@pabloCabrera7
Copy link

pabloCabrera7 commented Jun 19, 2020

Hola Miguel, estuve viendo y analizando el código, creo que esta llamada al hook de useGif ( y la siguiente linea de codigo) es innecesaria ya que al invocar a este hook, estamos realizando una nueva llamada a la API de gif que podríamos evitar, y que además muy probablemente el detalle del gif que queremos puede que no esté entre estos gif. Y asi también nos ahorramos la siguiente linea donde buscamos el gif.
Sería mejor simplemente realizar una llamada a la api con la id del gif del cual queremos saber el detalle, no ? O usar el useContext para obtener los gifs que ya tenemos sin hacer una nueva llamada a la API. ?
Espero haberme explicado.
Gracias y saludos.
@midudev

const {gifs} = useGifs()

@midudev
Copy link
Owner

midudev commented Jun 23, 2020

¡Me lo miraré! Gracias por comentarlo Pablo! 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants