Closed RodrigoCSolari closed 2 months ago
[pebble] "Enjoy your buy" me suena medio cavernícola. "Enjoy your purchase" estaba bien como frase para mostrar al usuario. El resto de los cambios están bien.
purchase
es más correcto que buy
pero entiendo que buy
es menos formal también. No sé si esto es de esas cosas que pide Blaze. Yo dejaría purchase 😋
[mountain] Conflictos con dev
que impiden hacer el merge
[pebble] "Enjoy your buy" me suena medio cavernícola. "Enjoy your purchase" estaba bien como frase para mostrar al usuario. El resto de los cambios están bien.
purchase
es más correcto quebuy
pero entiendo quebuy
es menos formal también. No sé si esto es de esas cosas que pide Blaze. Yo dejaría purchase 😋
listo lo volvi a purchase
[mountain] Conflictos con
dev
que impiden hacer el merge
conflictos solucionados
[pebble] Si vamos a utilizar finalmente purchase
, ¿tiene sentido que hagamos un cambio para modificar el nombre de la función handlePurchase
por handleBuy
? ¿Cuál sería el objetivo?
[pebble] Si vamos a utilizar finalmente
purchase
, ¿tiene sentido que hagamos un cambio para modificar el nombre de la funciónhandlePurchase
porhandleBuy
? ¿Cuál sería el objetivo?
Entiendo que se interpretó al principio de cambiar Purchase
por buy en todos lados, sin tener en cuenta que Enjoy your buy
no tiene mucho sentido. Siendo que se cambió en todos lados y que ya se corrigió lo verdaderamente importante, considero que esta corrección, si bien es correcta, no amerita corrección, con lo que procedo a aprobar. Si más adelante vemos que genera confusión, se arreglará
Hice un par de cambios menores. Hacete un pull antes de arrancar
[pebble] "Enjoy your buy" me suena medio cavernícola. "Enjoy your purchase" estaba bien como frase para mostrar al usuario. El resto de los cambios están bien.
No se si corresponde algún tag, pero hay algún conflicto con dev que hay que corregir antes de mergear