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

15.0 mig account cashbox #363

Open
wants to merge 4 commits into
base: 15.0
Choose a base branch
from

Conversation

juanpgarza
Copy link
Contributor

No description provided.

@roboadhoc
Copy link
Contributor

@juanpgarza
Copy link
Contributor Author

@jjscarafia Buenas! estaré muy agradecido si pueden dedicarle un tiempo a este PR.

@jjscarafia
Copy link
Member

buenas juan pablo! la verdad es que nos ibamos a saltear v15 para este módulo. ando medio hasta las manos para hacer review pero siempre que runbot esté verde tampoco tengo demasiado problema en agregarlo.
ideal review de alguien mas de la comunidad (pero eso no suele pasar jeje)

lo que si te pido es si podes ver la forma de mantener el historial (usando el approach por ej. de oca para llevar un modulo de un branch a otro). De esa manera, deberían figurar los commits originales de 16 mas un commit especifico por lo que vos adaptes.

Al hacerlo así, va a ser mucho mas facil hacer review porque lo único que reviso es ese commit de diferencia para entender que fue necesario adaptar a 15. Me explico? lo vez factible? Si hacemos eso, hago review técnico de ese commit y mezclo

@juanpgarza
Copy link
Contributor Author

Buenas @jjscarafia !!

Ok, si se entiende lo que planteas y estoy de acuerdo con vos de que es la forma correcta de hacerlo.
Lo hago así entonces.

Muchas gracias!

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