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

[16.0] [ADD] l10n_ve_pad: Provincias y municipios de Venezuela. #23

Open
wants to merge 3 commits into
base: 16.0
Choose a base branch
from

Conversation

rrebollo
Copy link

@rrebollo rrebollo commented Oct 16, 2024

Provincias y municipios de Venezuela.

  • Añade los estados actuales de Venezuela.
  • Añade los municipios actuales de cada estado.
  • Ajusta el formato de direcciones general para el país.

Los datos han sido procesados, pero fueron originalmente tomados del sitio oficial del Instituto Nacional de Estadística de Venezuela (INE) en la siguiente URL:

Además se complementaron con el trabajo de varios esfuerzos de proyectos en Github como:

@macagua
Copy link

macagua commented Oct 17, 2024

@rrebollo I am working on preparing the QA tests for the 16.0 branch for l10n to Venezuela, to merge into the OCA repository, including the process for the hr y payroll modules on this repository https://github.com/macagua/Closter_l10n_ve

@rrebollo
Copy link
Author

@macagua great, I'm available to cooperate. I tried to contact @lapinzon who has work merged to this repository but maybe he is a little busy. So, what to U think about this PR? Maybe a review???

@pedrobaeza
Copy link
Member

Aquí podemos hablar en español, como es un repositorio de localización. ¿Es necesario añadir a Odoo los distritos? ¿Es una información que se requiera para alguna operación financiera o fiscal? Lo digo por no recargar con más datos si no es necesario.

@rrebollo
Copy link
Author

@pedrobaeza @lapinzon originalmente no iba a incluir los districts pero es que en todas las implementaciones no formales que he visto están. Sin embargo los clientes que he tenido jamás los han usado para nada relevante, pero no he tenido un número tan grande de clientes, ¿ @macagua que crees?

@rrebollo
Copy link
Author

El punto distintivo del enfoque que usé:

  • almacenar los municipios en el modelos res.city (otras implementaciones creaban un modelo dedicado para esto)

@pedrobaeza
Copy link
Member

Sí, eso es importante, ya que existe ese modelo, que se use. Tal vez hay que cambiar a nivel de país la casilla para forzar que se usen ciudades, y así aparezcan seleccionables.

@rrebollo
Copy link
Author

Sí, eso es importante, ya que existe ese modelo, que se use. Tal vez hay que cambiar a nivel de país la casilla para forzar que se usen ciudades, y así aparezcan seleccionables.

Si entiendo bien creo que ya lo he hecho:

@pedrobaeza
Copy link
Member

Genial, Rolando! A ver si los demás nos confirman acerca de los distritos.

@rrebollo
Copy link
Author

rrebollo commented Nov 5, 2024

@pedrobaeza he podido checar con equipos contables venezolanos, la información de la parroquia no se usa para nada relevante desde el punto de vista contable. Me he traído un commit prescindiendo de todo lo relacionado con el feature.

@pedrobaeza pedrobaeza added this to the 16.0 milestone Nov 5, 2024
@pedrobaeza
Copy link
Member

Bueno, realmente puedes hacer squash de ambos commits para evitar un ruido innecesario, ¿no? ¿Puede entonces algún revisor de allí revisar y aprobar el PR y procedemos a fusionarlo?

…es, municipalities and districts)

[IMP] l10n_ve_pad: remove district management feature, set proper address format for country

[IMP] l10n_ve_pad: remove parroquias from module name in manifest

[IMP] l10n_ve_pad: add translation po template
@rrebollo rrebollo force-pushed the 16.0-ADD-l10n_ve_pad branch from 4f7af43 to 623c4e7 Compare November 5, 2024 10:54
@rrebollo
Copy link
Author

rrebollo commented Nov 5, 2024

@pedrobaeza hice el squash, trataré de encontrar un revisor.

@rrebollo rrebollo changed the title [16.0] [ADD] l10n_ve_pad: venezuelan Political-Administrative Division (states, municipalities and districts) [16.0] [ADD] l10n_ve_pad: Provincias y municipios de Venezuela. Jan 3, 2025
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