Skip to content

Feat/contributors section#76

Merged
TalkySafe143 merged 7 commits intoCapituloJaverianoACM:developfrom
arielLevita:feat/contributors-section
Aug 17, 2025
Merged

Feat/contributors section#76
TalkySafe143 merged 7 commits intoCapituloJaverianoACM:developfrom
arielLevita:feat/contributors-section

Conversation

@arielLevita
Copy link
Contributor

Creación de Sección "Contributors" en la página principal del sitio.

[AGREGADO] contributor.model.ts
En este archivo van a encontrar dos modelos: 1) el de usuario por defecto de GitHub y 2) el usuario que utiliza el formato que lleva el resto de los modelos. La idea con esto es poder transformar los objetos apenas traídos de GitHub para luego manejar únicamente el modelo estándar del resto de la aplicación y evitar futuro errores.

[AGREGADO] contributor.controller.ts
Este archivo consta de tres funciones:

  1. Una función que mapea un array de repositorios. Ustedes mencionaron que existe la posibilidad de agregar más repositorios, por lo que esta me pareció la forma más sencilla de hacerlo escalable y mantenible. Si van a utilizar una fuente que no sea GitHub, tendrían que cambiar esto por URLs completas.
  2. Una función asíncrona que se utilizará en el fetch para adaptar el formato de usuario que proviene de GitHub al estándar del resto del sitio web.
  3. Una función asíncrona que:
    a) llama a la función de mapear los repositorios
    b) llama a la API y trae los repositorios de GitHub
    c) combina los arrays de los colaboradores
    d) elimina los duplicados
    e) devuelve un array de colaboradores sin duplicados

[AGREGADO] contributors.tsx
Llamada y mapeo de data. Sólo avatares con sus respectivos links a perfiles de GitHub.
Utilicé estilos de TailwindCSS y del archivo global.css

[MODIFICADO] page.tsx
Aguegué el componente antes del Footer.

[MODIFICADO] next.config.ts
Agregué la API de GitHub a los dominios de imágenes permitidos.

A TENER EN CUENTA:

  • Si el tráfico del sitio va a crecer considerablemente, considerar utilizar otra estrategia para el fetch de datos que no vaya a saturar las llamadas a la API de GitHub.

@TalkySafe143 TalkySafe143 merged commit 902763b into CapituloJaverianoACM:develop Aug 17, 2025
1 check failed
@TalkySafe143
Copy link
Member

No hay que preocuparse por el workflow, estaba dando error por unas variables de entorno del repositorio.

Muy buen trabajo, quedó bastante bien.
Cambié un poco el tamaño, pero en general esta muy bien.

¡Muchas gracias!

@arielLevita
Copy link
Contributor Author

¡Hola!
Sí... pude ver que faltaba la URL de la API, pero no era algo que pudiera resolver (ni me correspondía jeje).
Me alegro de que les sirva el trabajo :-)
¡Éxitos con el proyecto!

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.

Colocar los contribuidores de los proyectos en el home

2 participants