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

[17.0][MIG] l10n_es_aeat_mod347: Migration to version 17.0 #3489

Merged
merged 86 commits into from
Mar 27, 2024

Conversation

ramiadavid
Copy link
Contributor

No description provided.

Borja L.S and others added 30 commits March 22, 2024 13:25
- Se han renombrado los módulos para usar la nomenclatura propuesta
  por OpenERP: l10n_es para el módulo base de localización (plan de
  cuentas), l10n_es_* para el resto de módulos.

- Se eliminan los módulos extra_addons/* que deberían moverse a
  los extra-addons genéricos (no son específicos de España).

- Se renombran los __terp__.py por __openerp__.py
…ciertos módulos, por los correspondientes que se modificaron para esta versión 6.0 y en ciertos __init__ adaptamos los imports a los nuevos nombres de los ficheros. Renombrado de los archivos de traducción españoles de es_ES.po a es.po y pequeñas refactorizaciones
* FIX: Renombrado de los archivos de traducción catalanas de ca_ES.po a ca.po
* IMP: Añadidos avisos NO ADAPTADO TODAVÍA A LA VERSIÓN 6.0 a varios módulos, limpieza l10n_es_partner_mercantil
…c module for aeat models, 347 module was portedto v6.0 and adds new module to print AEAT model 349.
…ueñas mejoras generales (vistas, traducciones, código) y corrección pequeño bug por un olvido en la adaptación de la v5 a la v6
… módulo del 347, que incluían los cambios hechos por la AEAT para la declaración 2010, para la versión 6.0 del módulo, además corregimos un bug encontrado por Jordi en la versión 5.0 hoy. También añado una nueva comprobación de los registros de empresas, ya que el cif de la empresa es requerido, por lo que no va a dejar confirmar el informe, mientras no se rellene este campo en todos los registros de empresa. La agrupación de pagos en efectivo se dejó igual que en la 5.0, ya que se sigue comportando como en la 5.0, sigo animando a que si alguien conoce como mejorarla adelante.
… las mejoras introducidas en el anterior commit
…recía Pexego en la licencia, quedan los compartidos.
…elo con el detalle de los trimestes obligatorio para la declaración del 2011.
…r Cif/Nif en el cálculo del 347, por si hay más de una empresa con el mismo cif..., pequeña refactorización usando rec_name en lugar de name_get
… los nuevos strings después de los recientes cambios en el 347.
…as. Necesario para las Sociedades Irregulares, Sociedades Civiles y Comunidad de Bienes sin regimen de módulos en el IRPF. Al resto de las empresas no les afecta ya que no facturan con IRPF. l10n_es_aeat_mod340: Correción de los autores en el modelo 340
…ron marcados como no instalables mientras no se compruebe que funcionan o migren, para poder usarlos o probarlos se debería poner el atributo installable de los ficheros __openerp__.py de cada módulo a True
[FIX]: Add security definition for objects l10n_es_aeat_mod347_calculate_records and l10n_es_aeat_mod347_export_to_boe.
[FIX]: Changed types to orm.Model, orm.TransientModel and orm.AbstractModel.
[FIX]: Contributions have been written in the standard format of the community.
[FIX]: Remove 'init_xml' keys, because it's no longer needed in v7.
[FIX]: Remove __author__ variables in files, because authors are put on manifest file (__openerp__.py).
[FIX]: Increased compatibility with standard PEP8.
[FIX]: Imports in standard format.
[IMP]: Add "state" field in tree view.
[IMP]: Option to generate report suppliers only.
[FIX]: Update address partner search to V7.
[FIX]: Update state in address partner to V7.
[FIX]: Update dependencies, now required the module documents to view generated file in V7.
[FIX]: Translate .pot file to v7.
[FIX]: Translate field "only_supplier" to spanish language.
[FIX]: Format the text that explains the module.
[IMP]: Add icon and screencapture.
- Cambiado flag de los partners para no incluirlos en el 347 (opción menos común), que incluirlos (que suelen ser todos).
- Eliminado flag para incluir los diarios, ya que no se activa por defecto y el usuario se confunde con los resultados. Se cogen todos los diarios de caja.
- Otras mejoras menores.
…ds, variables, classes and translations, and related PEP8 fixes.
@ramiadavid ramiadavid mentioned this pull request Mar 22, 2024
47 tasks
@ramiadavid ramiadavid marked this pull request as ready for review March 22, 2024 16:48
@pedrobaeza
Copy link
Member

/ocabot migration l10n_es_aeat_mod347

David, veo que te has puesto pronto a darle, jeje. ¿Te veremos este año por Tenerife? ¡Este año no te lo puedes perder!

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 22, 2024
l10n_es_aeat_mod347/data/tax_code_map_mod347_data.xml Outdated Show resolved Hide resolved
l10n_es_aeat_mod347/models/mod347.py Outdated Show resolved Hide resolved
l10n_es_aeat_mod347/views/mod347_view.xml Outdated Show resolved Hide resolved
@ramiadavid ramiadavid force-pushed the 17.0-mig-l10n_es_aeat_mod347 branch 2 times, most recently from a685d49 to 4c19801 Compare March 22, 2024 18:41
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aparte de los comentarios de código, revisa esta imagen con más comentarios:

imagen

l10n_es_aeat_mod347/views/mod347_view.xml Outdated Show resolved Hide resolved
l10n_es_aeat_mod347/views/mod347_view.xml Outdated Show resolved Hide resolved
l10n_es_aeat_mod347/data/tax_code_map_mod347_data.xml Outdated Show resolved Hide resolved
l10n_es_aeat_mod347/images/l10n_es_aeat_mod347.png Outdated Show resolved Hide resolved
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gracias!

@HaraldPanten queréis vosotros revisarlo?

Copy link
Contributor

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisión funcional en runboat

@HaraldPanten
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-3489-by-HaraldPanten-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 57aa581 into OCA:17.0 Mar 27, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at e10b744. Thanks a lot for contributing to OCA. ❤️

@ramiadavid ramiadavid deleted the 17.0-mig-l10n_es_aeat_mod347 branch March 27, 2024 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.