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

Tipografía #30

Merged
merged 18 commits into from
Dec 3, 2018
Merged

Tipografía #30

merged 18 commits into from
Dec 3, 2018

Conversation

mikelaranzabe
Copy link
Collaborator

No description provided.

@mikelaranzabe mikelaranzabe added this to the Superficie milestone Nov 28, 2018
@auto-comment
Copy link

auto-comment bot commented Nov 28, 2018

Acabas de iniciar un pull request para integrar los cambios que has ido realizando en el branch de vuestro tema, ¡bien! 👍

Recuerda asignar los metadatos correctamente (a la derecha):

  • En Reviewers, selecciona a las demás personas que han trabajado contigo en esta unidad
  • En Milestones, asigna este pull request al borrador correspondiente (Estrategia, Alcance, Estructura, Esqueleto, Superficie)

Ahora podéis realizar los comentarios necesarios sobre estas modificaciones aquí debajo, además de seguir realizando cambios al branch de vuestro tema. Cuando todos hayáis aprobado los cambios, la persona encargada de Comunicación deberá marcar este pull request como terminado, y entonces la profesora humana podrá hacer merge e integrar las modificaciones en el repositorio central.

Como siempre, si tenéis dudas, mencionad a Miren en vuestros comentarios.

@mikelaranzabe mikelaranzabe changed the title Añadir esquema tipografia Tipografía Nov 28, 2018
@jcantero7 jcantero7 removed their request for review November 29, 2018 16:30
meganeba
meganeba previously approved these changes Nov 30, 2018
@meganeba
Copy link
Collaborator

meganeba commented Dec 1, 2018

@jcantero7 @mikelaranzabe No tenemos los acentos en mi ordenador, puedes revisar mi parte por favor? Y borrar mi nombre al lado del titulo en el mismo tiempo? Muchas gracias chicos :)

@mirenbz
Copy link
Contributor

mirenbz commented Dec 3, 2018

Cuando tengáis el contenido listo, @meganeba y @jcantero7 , tenéis que seleccionar "Approve" en la pestaña de revisión de código para poder hacer merge.

Copy link
Contributor

@mirenbz mirenbz left a comment

Choose a reason for hiding this comment

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

Os falta añadir las referencias.

@mirenbz mirenbz merged commit 291b8ab into master Dec 3, 2018
@teacherbot
Copy link

teacherbot bot commented Dec 3, 2018

🎉 ¡Enhorabuena! Tus cambios han sido integrados en el repositorio principal Elementos de la Experiencia de Usuario.

Ahora tienes algunas tareas que hacer:

  • Una vez que ya se han integrado vuestros cambios, puedes eliminar el branch donde los habéis ido generando. Simplemente haz clic en el botón Delete branch aquí abajo y se borrará el branch de vuestro tema en este repositorio.
  • El repositorio original ahora ha cambiado (con los cambios que se acaban de integrar). Acuérdate de actualizar tu copia local (git pull origin master) y de borrar el branch de trabajo también de tu ordenador (git branch -d TEMA.)

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

Successfully merging this pull request may close these issues.

4 participants