Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Comments

Navbar#62

Merged
EGAMAGZ merged 5 commits intobinarybrains-club:mainfrom
AaronVanScoit:Navbar
Jun 18, 2025
Merged

Navbar#62
EGAMAGZ merged 5 commits intobinarybrains-club:mainfrom
AaronVanScoit:Navbar

Conversation

@AaronVanScoit
Copy link
Contributor

@AaronVanScoit AaronVanScoit commented Jun 12, 2025

Issue que soluciona: #

Explica qué modificaciones introduces con este pull request y por qué son necesarias, de manera
concisa y comprensible.

Considera que debe tener:

  • El proposito de los cambios
  • Que problemas resolvio o que nuevas funciones agrega
  • Menciona que cambios hay en la estructura del codigo
  • Enumera los cambios

En caso de ser un cambio relacionado con la pagina, agrega capturas del

Antes:

Despues:

Adicional

  • ¿Hay algún aspecto particular que los revisores deban tener presente al analizar este pull
    request?

  • ¿Existen recursos, documentación o referencias que puedan ser útiles durante la revisión?

  • ¿Esta solicitud incluye pruebas, ejemplos o una forma de validar los cambios realizados?

Lista de verificación

  • Pruebas superadas
  • Documentación actualizada

Gracias por ver esta solicitud, cualquier observación se tomará en cuenta :)

El proposito de mi pull request fue realizar el navbar

@EGAMAGZ
Copy link
Member

EGAMAGZ commented Jun 13, 2025

Por favor sigue las convencion de conventional commits para nombrar tu PR y formatea tu codigo haciendo uso de deno fmt

@EGAMAGZ
Copy link
Member

EGAMAGZ commented Jun 17, 2025

Desconozco si sigas trabajando en el responsive design como llegamos a comentar, pero favor de considerar esto mismo para que en pantallas chicas o medianas se visualize correctamente 😅

imagen

<BaseLayout title="Hacktion">
<main class="">
asd
pene
Copy link
Member

Choose a reason for hiding this comment

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

Quita esto @AaronVanScoit XD

@EGAMAGZ EGAMAGZ merged commit 70fb993 into binarybrains-club:main Jun 18, 2025
1 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants