Skip to content

Feat issue 8 - #91

Open
thegreatyamori wants to merge 5 commits into
javascriptecuador:masterfrom
thegreatyamori:feat-issue-8
Open

Feat issue 8#91
thegreatyamori wants to merge 5 commits into
javascriptecuador:masterfrom
thegreatyamori:feat-issue-8

Conversation

@thegreatyamori

@thegreatyamori thegreatyamori commented Mar 25, 2021

Copy link
Copy Markdown
Contributor

Descripción del cambio:

  • Diseño de la sección de blogs
  • Se diseño un pequeño algoritmo para distribuir las blog-cards dinámicamente.
  • Se utilizo styled components para facilitar la distribución de las blog-cards.
  • De acuerdo a una determinada estructura se pueden generar las blog-cards.

Razón del cambio:
Es una nueva feature solicitada para incluir en la pagina principal y la pagina de blog.

Captura:
image
image
image

Otros cambios:

  • Se utilizó el patrón Factory para generar diferentes tipos de cards, es compatible con blog-cards, city-cards y event-cards.
  • Se puede utilizar esta implementación para generar cards de ciudades y eventos con una misma clase.

Link al origen de este PR:
Add improvements to #8

@thegreatyamori

Copy link
Copy Markdown
Contributor Author

Descripción del cambio:

  • Se mejoró la consistencia del diseño según el prototipo de figma.
  • Se añadió la consulta de blogs utilizando graphql, sort desc por fecha y limite de 5 items.
  • Se reestructuró la organización de la carpeta blog, para poder incluir imágenes.
  • Se añadió los enlaces a cada entrada respectivamente.

Cosas que faltan:

  • Consultar la entrada más leída.
  • Actualizar una entrada con la explicación de como se construyó esta sección.
@rpalaciosg rpalaciosg mentioned this pull request Mar 29, 2021
5 tasks

@rpalaciosg rpalaciosg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Excelente trabajo 🎉 @thegreatyamori, puedes corregirle esa parte del url al renderizar el articulo que esta en la pagina src/pages/blog.js para aprovar los cambios y hacer el merge.

Comment thread src/pages/blog.js Outdated
@rpalaciosg

rpalaciosg commented Apr 5, 2021

Copy link
Copy Markdown
Member

@thegreatyamori se hizo el squash and merge de #90 puedes hacer un git pull --rebase para que dejes los cambios de este PR al final y tengas esta rama al dia y se resulevan los conflictos.

Me avisas si el articulo del blog lo vas a agegar desde este mismo PR. 🐱‍👤🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants