search

OCA / l10n-argentina

Odoo modules for Argentina
GNU Affero General Public License v3.0
22 stars 51 forks source link

[WIP] 11.0-mig-l10n_ar_* #50

Closed Maartincm closed 1 year ago

oca-clabot commented 6 years ago

Hey @Maartincm, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet. You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla Here is a list of the users:

Appreciation of efforts, OCA CLAbot

pedrobaeza commented 6 years ago

Es mejor poner un módulo por PR, y además más fácil de revisar, corregir errores de Travis, etc.

Maartincm commented 6 years ago

Todavía falta bastante trabajo, pero vamos a intentar ordenar los PR de esa manera.

sebastiken commented 5 years ago

@pedrobaeza buenas, hay varios commits que impactan en varios módulos al mismo tiempo. Sería un muy arduo el trabajo de armar un módulo por PR, además de que incurriría en tener que reescribir la historia para segmentar los commits. Se me ocurre que lo que se puede hacer es separar en PRs los commits que comienzan con la etiqueta [MIG] que mayormente son módulos completos y luego los últimos commits puede ser que tengamos que armarlos en un mismo PR pero ya no sería por módulo. Si estás de acuerdo, eso nos ayudaría a nosotros a poder avanzar más rápido.

Muchas gracias. Saludos.

pedrobaeza commented 5 years ago

@sebastiken OK, pero esto es lo que pasa por acumular en el mismo PR todo eso desde el 24 de agosto... Mejoras incrementales y metodología ágil es mejor :wink:

sebastiken commented 5 years ago

Gracias Pedro. Igual este PR lo cerramos y creamos otros por cada módulo migrado según el esquema planteado en el comentario anterior. Seguramente una vez hecho esto y todo mergeado, avanzaremos en mejoras incrementales.

Gracias.

pedrobaeza commented 5 years ago

Lo digo más que nada porque si no poner en verde esto os va a costar un montón.

sebastiken commented 5 years ago

Lo digo más que nada porque si no poner en verde esto os va a costar un montón.

En eso tenés razón. Vamos a armar los PR lo mejor posible.

sebastiken commented 5 years ago

@pedrobaeza buenas, cometí un error e hice un Pull Request de la forma que habíamos dicho aquí que no íbamos a hacer. Porque la idea era hacer un PR por módulo. El tema que ya lo mergeé con la rama 11.0. ¿Vos podrás volver atrás esa rama a este commit 347341ba4436dead5c6159f26e804fb1f6b740ef para hacer luego los PR por cada módulo? Estamos en este proceso de mergeo de muchos módulos y lo queremos realizar de la manera que lo planteaste. Gracias.

pedrobaeza commented 5 years ago

Pero está en la rama principal 11.0?

sebastiken commented 5 years ago

Sí, pero es hice el PR y lo mergeé sin querer pensando que estaba en mi fork. Yo tengo permisos en este repo también.

pedrobaeza commented 5 years ago

OK, ya lo tienes.

sebastiken commented 5 years ago

Muchísimas gracias @pedrobaeza !