Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat issue 8 #91

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

thegreatyamori
Copy link
Contributor

@thegreatyamori thegreatyamori commented Mar 25, 2021

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
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
Copy link
Member

@rpalaciosg rpalaciosg left a comment

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.

src/pages/blog.js Outdated Show resolved Hide resolved
@rpalaciosg
Copy link
Member

rpalaciosg commented Apr 5, 2021

@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants