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

init #10

Closed
wants to merge 4 commits into from
Closed

init #10

wants to merge 4 commits into from

Conversation

lapinzon
Copy link

Feliz nuevo año.

Este módulo obtiene tasa de cambio desde el Banco Central de Venezuela

@lapinzon
Copy link
Author

lapinzon commented Jan 7, 2024

@pedrobaeza , espero haya tenido unas bonitas fiestas.

Pedro, podría apoyarme y orientarme para intentar reactivar este proyecto? Por ahora quisiera agregar este módulo.

Gracias

@pedrobaeza pedrobaeza added this to the 16.0 milestone Jan 8, 2024
@pedrobaeza
Copy link
Member

Hola, feliz año.

Veo que aunque el contenido del módulo parece correcto, habría que poner un poco de orden en la parte formal:

  • Por un lado, si tienes que actualizar pre-commit y demás, hazlo por favor usando copier y actualizando la plantilla (copier update -f --trust) en lugar de hacerlo a mano, y en un PR separado.
  • Por otro, una vez el primero esté fusionado, haz otro PR (o reutiliza éste), para añadir el módulo con un solo commit (ya que cada commit aquí no está añadiendo valor), y siguiendo las convenciones de mensaje de commit: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message, lo que sería poner algo así como [ADD] res_currency_rate_provider_BCV: Obtain rates from BCV
  • Igualmente, pon un título de PR adecuado: [16.0][ADD] res_currency_rate_provider_BCV: Obtain rates from BCV

Un saludo.

@lapinzon lapinzon closed this Jan 9, 2024
@lapinzon lapinzon deleted the bcv_provider branch January 9, 2024 03:16
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