Skip to content

Conversation

@DavidVargas-Ctrl
Copy link
Contributor

correciones de @Samu-Kiss

@Samu-Kiss
Copy link
Contributor

Ahora siii :)

@Samu-Kiss
Copy link
Contributor

bro, @DavidVargas-Ctrl, me acabo de dar cuenta que el footer en móvil está rarito, yo creo que lo mejor ahí es dejar centrados los elementos. Y los contribuidores sería bueno ponerle el titulito, ese iría en montserrat,

image

@TalkySafe143
Copy link
Member

Finalmente en que quedo este PR? Estará en Draft mientras

@TalkySafe143 TalkySafe143 marked this pull request as draft September 27, 2025 01:04
@Samu-Kiss
Copy link
Contributor

Finalmente en que quedo este PR? Estará en Draft mientras

Hola bro, ya está casi todo. Falta corregir lo de movil

@adrianrrruiz
Copy link
Contributor

@DavidVargas-Ctrl creo que tienes algo pendiente por acá

@DavidVargas-Ctrl
Copy link
Contributor Author

listo, lo corregiré entonces

@adrianrrruiz
Copy link
Contributor

Como vas? @DavidVargas-Ctrl

@DavidVargas-Ctrl
Copy link
Contributor Author

Lo trabajo el fin de semana, entre semana he estado un poco lleno de cosas

@DavidVargas-Ctrl
Copy link
Contributor Author

Listo, ya quedo corregido

@DavidVargas-Ctrl DavidVargas-Ctrl marked this pull request as ready for review October 5, 2025 13:40
@Samu-Kiss
Copy link
Contributor

Listo, ya quedo corregido

epa ya reviso

@Samu-Kiss Samu-Kiss self-requested a review October 5, 2025 15:53
Copy link
Contributor

@Samu-Kiss Samu-Kiss left a comment

Choose a reason for hiding this comment

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

Image

bro, yo creo que es mejor quitarle ese mini texto que sale de los nombres, no es legible

@adrianrrruiz
Copy link
Contributor

Image bro, yo creo que es mejor quitarle ese mini texto que sale de los nombres, no es legible

@DavidVargas-Ctrl

@DavidVargas-Ctrl
Copy link
Contributor Author

Bueno, yo lo quito ;b

@DavidVargas-Ctrl
Copy link
Contributor Author

Ya lo quite
cat-thinking

Copy link
Contributor

@Samu-Kiss Samu-Kiss left a comment

Choose a reason for hiding this comment

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

Nice, gracias bro

@Samu-Kiss
Copy link
Contributor

@Achalogy @adrianrrruiz porfa alguno revise porfis :)

@Samu-Kiss
Copy link
Contributor

btw el merge no debería ser a develop?

@TalkySafe143 TalkySafe143 changed the base branch from main to develop October 7, 2025 00:02
@adrianrrruiz
Copy link
Contributor

Por favor solucionar conflictos para revisar la versión final @DavidVargas-Ctrl. Intenté mergearlo, pero siento que dañe varias cosas, entonces mejor no completé el merge.
image

@Samu-Kiss
Copy link
Contributor

Por favor solucionar conflictos para revisar la versión final @DavidVargas-Ctrl. Intenté mergearlo, pero siento que dañe varias cosas, entonces mejor no completé el merge. image

ps bro yo ya los tengo solucionados, pero no estoy seguro, a ver se ve bien, pero no se

@adrianrrruiz
Copy link
Contributor

Por favor solucionar conflictos para revisar la versión final @DavidVargas-Ctrl. Intenté mergearlo, pero siento que dañe varias cosas, entonces mejor no completé el merge. image

ps bro yo ya los tengo solucionados, pero no estoy seguro, a ver se ve bien, pero no se

Para estoy toca que traigas los cambios de develop y revises que no se te haya dañado nada. Es que cuando traje cambios se dañó eso que dije, por eso no cerré el commit.

@Samu-Kiss
Copy link
Contributor

Por favor solucionar conflictos para revisar la versión final @DavidVargas-Ctrl. Intenté mergearlo, pero siento que dañe varias cosas, entonces mejor no completé el merge. image

ps bro yo ya los tengo solucionados, pero no estoy seguro, a ver se ve bien, pero no se

Para estoy toca que traigas los cambios de develop y revises que no se te haya dañado nada. Es que cuando traje cambios se dañó eso que dije, por eso no cerré el commit.

ik, pero pues de lo visual no se daño nada, de lo otro no estoy muy seguro, todo parece funcionar...

@Samu-Kiss
Copy link
Contributor

@adrianrrruiz @Achalogy porfi revisen :3

@adrianrrruiz
Copy link
Contributor

Lo acabe de probar con lo último y aún tiene estos detalles:
image
@Samu-Kiss y @DavidVargas-Ctrl

@TalkySafe143
Copy link
Member

Voy a colocar esto en draft, nunca se arregló nada

@TalkySafe143 TalkySafe143 marked this pull request as draft January 7, 2026 07:05
@TalkySafe143 TalkySafe143 marked this pull request as ready for review January 7, 2026 07:49
@TalkySafe143 TalkySafe143 requested a review from Samu-Kiss January 7, 2026 07:49
@TalkySafe143 TalkySafe143 self-assigned this Jan 7, 2026
@TalkySafe143
Copy link
Member

Ya lo arreglé yo

Copy link
Contributor

@Samu-Kiss Samu-Kiss left a comment

Choose a reason for hiding this comment

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

CARAMELO

@TalkySafe143 TalkySafe143 merged commit 0aeeb7d into develop Jan 7, 2026
2 checks passed
@TalkySafe143 TalkySafe143 deleted the feature/footer-collabs branch January 7, 2026 18:17
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.

5 participants