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

Cambio-TontoDelDia #55

Merged
merged 7 commits into from
Apr 3, 2024
Merged

Cambio-TontoDelDia #55

merged 7 commits into from
Apr 3, 2024

Conversation

JLeonN
Copy link
Contributor

@JLeonN JLeonN commented Apr 1, 2024

No description provided.

francosang
francosang previously approved these changes Apr 2, 2024
const { cargaa } = useNombres();
const { carga } = useNombres();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hay que borrar estas dos líneas porque te dan conflictos

También se creó un HOOK para el componente y se agregó la URL a la parte de la API. 🛠️
Pequeño error de ubicación. 📍😊

const Inicio = () => {
const { cargaa } = useNombres();
const { tonto } = useObtenerTonto();
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hay algo raro acá. No aparecen mis cambios.

Fijate el PR que hice yo: #51

Por qué no aparecen mis cambios acá???

const { tonto } = useObtenerTonto();

if (cargaa) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vos mismo agregaste este componente de carga, ya no lo querés?

Por qué lo estás borrando?

No me descanse 😂

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capaz que no entendes completamente cómo funciona esto.

Cada hook tiene su variable de carga y sus datos.
Los hooks son independientes entre ellos y pueden estar los dos cargando a la vez.

Si vos querés mostrar un componente de carga para un hook tenes que usar la variable de carga de ese hook.

En este caso estabas usando la variable de carga de un hook y los datos de otro hook diferente.

@francosang
Copy link
Collaborator

Hace dos cosas:

  1. Actualiza tu rama con main para traerte los últimos cambios
  2. Volvé a poner el componente de carga

@JLeonN JLeonN requested a review from francosang April 2, 2024 22:03
@JLeonN
Copy link
Contributor Author

JLeonN commented Apr 2, 2024

Creo que ya quedó franco.

Me olvidé de borrar el componente que no se usa. 🤦‍♂️💻
Copy link
Collaborator

@francosang francosang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sabeee 👏 👏

@JLeonN JLeonN merged commit bf22312 into main Apr 3, 2024
1 check passed
@JLeonN JLeonN deleted the Cambios-TontoDelDia branch April 3, 2024 15:15
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

Successfully merging this pull request may close these issues.

2 participants