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

Error404 casi terminado. #12

Merged
merged 17 commits into from
Feb 22, 2024
Merged

Error404 casi terminado. #12

merged 17 commits into from
Feb 22, 2024

Conversation

JLeonN
Copy link
Contributor

@JLeonN JLeonN commented Feb 17, 2024

Solicité a Christian que creara una imagen para este componente; aún no estoy seguro si lo hará, pero mientras tanto, utilizaremos la que tenemos.

He creado una carpeta específica para almacenar el componente 404. Esto facilitará la organización, especialmente porque planeo agregar más contenido al error 404 en el futuro.
El componente de Error 404 está casi terminado.
Solo falta agregarle algunos detalles más, pero me he quedado sin ideas por el momento.
Solicité a Christian que creara una imagen para este componente; aún no estoy seguro si lo hará, pero mientras tanto, utilizaremos la que tenemos.
Por otro lado, en VSC me aparece un error en el componente del botón relacionado con los PROPS. No sé la razón exacta, pero a pesar de eso, sigue funcionando correctamente.
He terminado de pulir el error 404. Finalmente, opté por convertirlo en una tarjeta similar a la que muestra al 'Cowboy del día'
Aún persiste el problema en la tarjeta del día. Me vi obligado a agregar un condicional ('if') para validar la tarjeta. No estoy seguro si es la mejor solución, pero al menos ahora funciona. Sería conveniente revisar esta parte.
Corrigiendo errores ortográficos y otros detalles adicionales.
Hey Franco, necesito enviarte un mensaje de voz por WhatsApp o Discord, tengo un lío enorme. Más o menos he logrado resolverlo. Resulta que metí la pata con toda esta rama. La idea original era solo trabajar en el componente Error404, pero terminé modificando otras cosas para que ese componente quedara bien y, sin darme cuenta, terminé alterando casi todo, ¡ja! Bueno, luego te contacto por audio para explicarte mejor.
He realizado modificaciones en la página de inicio. Ahora, esta sección mostrará únicamente dos tarjetas: una destacando al 'tonto del día' y otra al 'cowboy del día'. Toda la información adicional estará disponible en sus respectivas secciones, como la lista completa de 'tontos del día' en la sección dedicada a ello.
Comment on lines 4 to 6
if (!button) {
return;
}
Copy link
Collaborator

@francosang francosang Feb 20, 2024

Choose a reason for hiding this comment

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

Este fue lo que te dijo ChatGPT?

Creo que no es necesario, me parece que se puede borrar.
No sería correcto crear un boton que no tiene ruta ni informacion.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Si eso mismo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Acabo de eliminarlo y no tuve ningún problema, tal como me mencionaste. Sin embargo, en el componente de la tarjeta del día, todavía tengo dificultades.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Franco, ya he solucionado el problema. Como mencionaste, un componente vacío puede generar un error. He comentado esa parte del código y eliminado el 'if', y ahora funciona correctamente.

return (
<>
<h2>Error 404</h2>
<div>
<TarjetaDia tarjeta={tarjeta} />
Copy link
Collaborator

Choose a reason for hiding this comment

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

Creo que para el error 404 no tendriamos que usar el componente Tarjeta.

El componente Tarjeta tiene propiedades exclusivas para un tonto, como un nombre y titulos y una cantidad.
Creo que seria mejor que esta pagina no use este componente y tenga todo el HTML directamente.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Al principio, estaba utilizando HTML plano, como mencionas. Después lo cambié, ¿por qué? Bueno, ¡ni yo mismo lo sé, jaja!


{/* Tarjeta del cowboy del día */}
<h3 className="tituloH3">Cowboy del día de hoy es...</h3>
<TarjetaDia />
Copy link
Collaborator

@francosang francosang Feb 20, 2024

Choose a reason for hiding this comment

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

Es mejor no usar el componente tarjeta si no va a tener propiedades y va a estar vacio.
De esta forma te evitas poner el if dentro del componente.

Ademas, no es un uso correcto para el componente.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

La idea era incluir los datos del Cowboy del día, pero lamentablemente no pude completarlo a tiempo. Ese es el plan. Pretendo que funcione de manera similar al Tonto del día, pero enfocado en el Cowboy destacado del día.

Después de eliminar los 'if', todo está funcionando correctamente. Tuve que comentar el componente 'TarjetaDelDia', ya que por ahora no se está utilizando en 'Inicio'. Sin embargo, ¿qué sucederá cuando comience a usar el componente 'TarjetaDelDia' en 'Inicio'? ¿Será necesario volver a utilizar el 'if'?
Cambio de nombre de componente a Tarjeta
He completado el Error 404! Espero que esta vez el componente esté listo pronto. Realicé varios cambios significativos, y debo admitir que me gusta más en su forma actual. De todos modos, si necesitas más ajustes, házmelo saber y seguimos trabajando en ello juntos.
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.

Go go goooo!

@JLeonN JLeonN merged commit 96fc7e6 into main Feb 22, 2024
1 check passed
@JLeonN JLeonN deleted the Error404 branch February 22, 2024 09:48
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