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

[IPL-288] Update conekta logo #107

Merged
merged 1 commit into from
Oct 14, 2022

Conversation

ElvisHeredia
Copy link
Contributor

@ElvisHeredia ElvisHeredia commented Sep 27, 2022

Detalle del PR

Se cambia por el logo nuevo de Conekta

Descripción

Debido a los cambios de imagen, se requiere cambiar el logo dentro del plugin de Prestashop

IPL-288

Motivación y Contexto

Tipos de Cambios

  • Cambio en la documentación / refactorización / actualización de dependencia
  • Corrección de errores (cambio que soluciona un problema)
  • Nueva característica (cambio que agrega funcionalidad)
  • Cambio de comportamiento (corrección o característica que cambiara la funcionalidad existente)

Checklist

  • Mi código sigue el estilo de codificación de proyecto.
  • Mi cambio requiere un cambio en la documentación.
  • He actualizado la documentación correspondiente.
  • Agregue los tests para cubrir mis cambios.
  • Todos los test nuevos y existen pasaron correctamente.
  • Actualice el changelog.

Evidencias

Antes:
image

Despues:

image

Convención de merge

Este repositorio usa Gitflow

Branch desde Branch hacia Merge commit Squash and Merge
feature/ develop
hotfix/ master - release/
fix/ release/x.x.x
release/x.x.x master
backport/ develop

¿Qué pasa si hago merge commit cuando debía hacer un squash?

  • Nada grave, simplemente nos estaríamos llevando commits innecesarios al branch donde mergeemos, ejemplo, si en mi branch feature/ tengo 15 commits y los comentarios no son descriptivos (fix, fix tests, working) me estaría llevando todos esos a develop, cuando con el Squash voy a llevar uno solo.

¿Qué pasa si hago Squash cuando debía hacer un merge commit?

  • En este caso no estaríamos cumpliendo la función que esperamos que es llevar los commits de un branch a otro para emparejarlos, por ejemplo, cuando mergeamos una release a master, el objetivo es dejar parejos develop y master, y si usamos squash como estrategia de merge van a seguir teniendo commits distintos, por lo que en el siguiente release podríamos tener conflictos.

@ElvisHeredia ElvisHeredia changed the title [Feat] Update conekta logo [IPL-288] Update conekta logo Sep 27, 2022
@ElvisHeredia ElvisHeredia force-pushed the feature/IPL-288-change-onekta-logo branch from 5b87261 to 162a514 Compare September 29, 2022 02:19
@ElvisHeredia ElvisHeredia force-pushed the feature/IPL-288-change-onekta-logo branch 2 times, most recently from 99eee58 to 26b978b Compare September 29, 2022 21:09
EricDominguez1
EricDominguez1 previously approved these changes Sep 29, 2022
@ElvisHeredia ElvisHeredia force-pushed the feature/IPL-288-change-onekta-logo branch 2 times, most recently from eafc362 to acd4638 Compare October 11, 2022 23:40
@ElvisHeredia ElvisHeredia force-pushed the feature/IPL-288-change-onekta-logo branch from acd4638 to b9ce30f Compare October 13, 2022 00:13
@ElvisHeredia ElvisHeredia merged commit eea9f87 into master Oct 14, 2022
@ElvisHeredia ElvisHeredia deleted the feature/IPL-288-change-onekta-logo branch October 14, 2022 17:59
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.

2 participants