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

[15.0][FIX+IMP] l10n_es_aeat_mod592: Compare if no weight with the correct decimal precision + Misc changes #3860

Open
wants to merge 4 commits into
base: 15.0
Choose a base branch
from

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Dec 19, 2024

Cambios hechos:

  • Comprar si el peso no es correcto con la precisión decimal correspondiente.
  • Definir digits a los campos kgs, no_recycling_kgs, product_uom_qty
  • Definir datos correctos en el excel que se exporta
  • Posibilidad de exportar csv
  • Posibilidad de exportación a BOE

Por favor @pedrobaeza puedes revisarlo?

@Tecnativa TT52290 TT52986

@pedrobaeza pedrobaeza added this to the 15.0 milestone Dec 19, 2024
@pedrobaeza
Copy link
Member

En el BOE exportado, ¿cuántos decimales se tienen? ¿Es la misma precisión?

@victoralmau
Copy link
Member Author

En el BOE exportado, ¿cuántos decimales se tienen? ¿Es la misma precisión?

Actualmente no hay configuración para exportar a BOE.
Quizás también sería recomendable definir en los campos correspondientes la precisión decimal de peso y mostrar el dato adecuado en pantalla

@pedrobaeza
Copy link
Member

Bueno, aunque no haya exportación, es importante saber cómo lo exportaría, porque ésa es la precisión que va a marcar el mínimo exportable, y la que habría que utilizar para todo.

@victoralmau
Copy link
Member Author

Veo que hay una opción de export_type=float en las líneas de exportación, aunque en todos se define decimal_size por lo que creo no es muy importante en este caso el como se defina el campo.

En cualquier caso he añadido el digits correspondiente también a los campos para que los datos que se muestren por pantalla sean coherentes.

@victoralmau victoralmau force-pushed the 15.0-fix-l10n_es_aeat_mod592-TT52290 branch from fbfd180 to 9ffbf69 Compare January 16, 2025 16:49
@victoralmau victoralmau changed the title [15.0][FIX] l10n_es_aeat_mod592: Compare if no weight with the correct decimal precision [15.0][FIX+IMP] l10n_es_aeat_mod592: Compare if no weight with the correct decimal precision + Misc changes Jan 16, 2025
@pedrobaeza
Copy link
Member

Entiendo que falta como mínimo la exportación a CSV, ¿verdad?

@victoralmau victoralmau force-pushed the 15.0-fix-l10n_es_aeat_mod592-TT52290 branch from 214bfe9 to cfa5e1c Compare January 20, 2025 08:28
… exporta

Cambios hechos:
- Se añade el concepto y el product_description al método _get_csv_report_info() del mixin (necesario en ambos)
- Se añade el campo product_description al mixin (necesario en ambos)
- Los valores de Kilogramos y Kilogramos No Reciclados se deben definir con , como separador decimal
- Definir el valor del campo Fecha Hecho Contabilizado en formato: DD-MM-YYYY
- Adquiriente: Se define como Descripción Producto la descripción del producto en lugar de fiscal_acquirer
- Adquiriente: Se añade la columna Régimen fiscal (fiscal_acquirer)  después de la descripción de producto
- Adquiriente: Se mueven los campo Prov./Dest. al lugar adecuado, antes de Kilogramos

TT52986
@victoralmau victoralmau force-pushed the 15.0-fix-l10n_es_aeat_mod592-TT52290 branch from cfa5e1c to a0b3a14 Compare January 21, 2025 07:57
@victoralmau victoralmau force-pushed the 15.0-fix-l10n_es_aeat_mod592-TT52290 branch from a0b3a14 to ebede0b Compare January 27, 2025 08:54
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