search

ReactJS-BA / sitio-web

MIT License
6 stars 10 forks source link

maquetado de la seccion de SOCIAL , react icons Install #2

Closed Miguel-Andres closed 3 years ago

Miguel-Andres commented 3 years ago

Hola , me faltaria linkear los iconos con las redes sociales correspondientes , Me interesa Tener el Diseño y los Assets para guiarme mas facil , Es mi primer PR en un proyecto remoto

rodrigo398 commented 3 years ago

Gracias por el aporte Luis. Felicitaciones por el primer PR 🎉 . Excelente iniciativa de agregar esos icons para social networks, son bastantes utiles para el sitio de ReactJS. Como ya lo mencionaste, seria genial tener los assets y designs para tener una guia y continuar laburando sobre eso. Esperemos a ver que opinan los demas sobre estos cambios antes de mergear.

nanovazquez commented 3 years ago

Hola @Miguel-Andres ! Podes encontrar el diseño aquí: https://www.figma.com/file/74vsJs9FtgDHbvJSWyS8Ui/React-BA?node-id=205%3A1

No tenemos que seguirlo 100% al pie de la letra, pero si respetar las definiciones de diseño.

rodrigo398 commented 3 years ago

@Miguel-Andres podrias por favor mergear los ultimos cambios de main a tu branch?, de esta manera podes tener este PR en sync con main, y ademas se corregirian algunos detalles con respecto a saltos de linea e indentacion tambien.

Una vez que hagas un pull a main, te recomiendo que hagas un npm install , porque se agregaron nuevos paquetes.

Avisame si necesitas una mano con eso y lo hacemos juntos.

Miguel-Andres commented 3 years ago

@rodrigo398 @leilaybnz Hola Gracias por las aclaraciones , intente hacer lo que me comentaron , estoy atento a cualquier otra corrección , si hay algo que no estoy teniendo en cuenta con respeto al manejo de github es mi primera vez colaborando en un proyecto en remoto . Asi que no estoy muy pulido con esta forma de colaborar PERO nada que GOOGLE no pueda Solucionar y tambien me sirve de mucho su feedback .........Gracias !

rodrigo398 commented 3 years ago

@Miguel-Andres gracias por los cambios. Faltarian algunos detalles minimos y esperar el feedback sobre los demas cambios que te solicitaron. No se bien si hay urls para los icons, quizas @leilaybnz podria ayudar con eso.

leilaybnz commented 3 years ago

@Miguel-Andres gracias por los cambios. Faltarian algunos detalles minimos y esperar el feedback sobre los demas cambios que te solicitaron. No se bien si hay urls para los icons, quizas @leilaybnz podria ayudar con eso.

Acá les mando las urls para los iconos: Twitter: https://twitter.com/React_BA Slack: slack://channel?team=T0434657X&id=C045DPHNR Meetup: https://www.meetup.com/React-en-Buenos-Aires/ YouTube: https://www.youtube.com/channel/UC8lMWXElwhflZxWqsE6BuPQ Email: reactjs.ba@gmail.com

rodrigo398 commented 3 years ago

PR revisado y mergeado. Los cambios faltantes se van a agregar en siguientes PRs.