Closed lapinzon closed 10 months ago
lapinzon
commented
10 months ago @pedrobaeza , olvidé preguntarle si aunque es para Venezuela, los comentarios de commits y PR deben ser en inglés, o incluso estos mensajes.
Le agradezco mucho su apoyo. Apenas estoy empezando a entender el proceso.
pedrobaeza
commented
10 months ago En los mensajes de commit, es conveniente mantener el inglés para que así pueda tener mayor audiencia técnica, aunque el README o la conversación en PRs pueda ser en español. Así lo hacemos al menos en l10n-spain.
lapinzon
commented
10 months ago Por lo demás, no sé si hay algún mantenedor allí en Venezuela que pueda validar funcionalmente.
Sobre el mantenedor no tengo conocimiento, pero por el abandono de esta localización supongo que no lo hay. Que debemos hacer en este caso?
pedrobaeza
commented
10 months ago No es necesario un nuevo PR. Haz los cambios, y con git commit --amend --all puedes mantener el commit, y luego git push -f para subirlo sobreescribiendo el contenido.
lapinzon
commented
10 months ago Gracias Pedro. Su apoyo ha sido muy valioso para mi. Este primer pequeño paso espero acompañarlo de nuevos aportes, sin embargo, no estoy seguro de cómo actualizar a l10n-venezuela. Parece que el proyecto original externo a OCA tiene 9 años sin actividad.
Pedro, después de este PR aprobado, que viene?
pedrobaeza
commented
10 months ago Viene fusionarlo de acuerdo a las reglas de https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#review, pero bueno, vamos adelante con ello:
/ocabot merge nobump
OCA-git-bot
commented
10 months ago This PR looks fantastic, let's merge it! Prepared branch 16.0-ocabot-merge-pr-12-by-pedrobaeza-bump-nobump, awaiting test results.
OCA-git-bot
commented
10 months ago Congratulations, your PR was merged at 251b8a1b4813359a0d570577c85335eb40f56ae7. Thanks a lot for contributing to OCA. ❤️
Obtiene tasa de cambio desde la página web del Banco Central de Venezuela.