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

Estado Global #75

Merged
merged 2 commits into from
Apr 23, 2024
Merged

Estado Global #75

merged 2 commits into from
Apr 23, 2024

Conversation

JLeonN
Copy link
Contributor

@JLeonN JLeonN commented Apr 21, 2024

Franco, esto medio que ya quedó, pero estoy teniendo unos problemas. La página no se está renderizando correctamente.

Franco, esto medio que ya quedó, pero estoy teniendo unos problemas. La página no se está renderizando correctamente.
@JLeonN JLeonN requested a review from francosang April 21, 2024 21:31
return tokenn;
};

export const EstadoGlobal = ({ contenido }) => {
Copy link
Collaborator

@francosang francosang Apr 22, 2024

Choose a reason for hiding this comment

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

Ya me di cuenta cual es el error.
Para encontrarlo hice lo que todos hacemos para encontrar errores: usar console.log() por todos lados 🤣.

Resulta que la variable contenido esta vacía, viene con null.
Si la mostras en un console.log("Contenido:", contendido) sale como "Contenido: null".

Esto es por culpa del nombre de la variable, yo pensé que podía tener cualquier nombre, pero no.
Hay que ponerle children, tiene que llamarse children si o si.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Asi que cambiale el nombre a children.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

listo gracias

{contenido} por {children}
@JLeonN JLeonN requested a review from francosang April 22, 2024 20:14
@JLeonN JLeonN merged commit 36adb12 into main Apr 23, 2024
1 check passed
@JLeonN JLeonN deleted the ContextoGlobal branch April 23, 2024 09:41
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