-
-
Notifications
You must be signed in to change notification settings - Fork 533
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
[10.0][add][l10n_es_mis_report_trial_balance] #789
[10.0][add][l10n_es_mis_report_trial_balance] #789
Conversation
ebcb016
to
707a525
Compare
8023c9a
to
e772798
Compare
'version': '10.0.1.0.0', | ||
'license': 'AGPL-3', | ||
'depends': [ | ||
'l10n_es_mis_reports', # OCA/account-financial-reporting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you have typo error.
Your other module is named 'l10n_es_mis_report' so without the final 's'
Same error into your XML data files, you reference the module with a 's'.
Or maybe the error come from your other MR who has a bad module name ( #788 don't forget to update your setup if needed).
55d2ef5
to
ad5ecf8
Compare
Hola @jbeficent , estoy preparando un entorno para probar los PR que tienes de mis_builder y ponernos al día con esto. Por el momento este módulo me da error por CPU time limit exceded, se que con mas maquina (o mejor configurado) lo solucionaría pero habría que ver si se puede o debe hacer de otra forma. |
Bueno, como decía, ampliando tiempo de CPU lo instala sin problemas, pero en una instalación dejada caer sin tocar configuración se queda a medio, no se si alguien pueda tener el mismo problema. |
Si, es normal. Pero en v11 este modulo no hara falta porque el sumas y
saldos jerarquico ya lo da el account_financial_report.
…On Sat, Mar 24, 2018, 9:41 PM Angel Moya - PESOL ***@***.***> wrote:
Bueno, como decía, ampliando tiempo de CPU lo instala sin problemas, pero
en una instalación dejada caer sin tocar configuración se queda a medio, no
se si alguien pueda tener el mismo problema.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#789 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHU_VmLqFEO7p5jvLNX2kVVO_vhYXgflks5thq9wgaJpZM4SZ8yP>
.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (based on code and installation)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Al final instalado y funcionando bien.
ad5ecf8
to
9a847c0
Compare
@angelmoya @pedrobaeza Teniendo en cuenta que está pendiente ampliar MIS Builder OCA/mis-builder#74 quitaré de este PR la plantilla que muestra los informes del sumas y saldos expandidos, y propondré un PR con el módulo l10n_es_mis_report_trial_balance_expanded. Cuando OCA/mis-builder#74 se complete cerraré el PR de los módulos l10n_es_mis_report_*_expanded. Por ahora son necesarios, hasta que el cambio en MIS builder se haga efectivo. |
9a847c0
to
e5bae56
Compare
He quitado el formato expandido de este módulo y lo he puesto en #824 |
Añade temporalmente OCA/mis-builder#78 en oca_dependencies.txt y así podemos probar éste como el definitivo |
de no-autoexpansión de cuentas.
@pedrobaeza He cambiado las plantillas para que se ajusten a OCA/mis-builder#78 |
@jbeficent esto no es algo que esté regulado para el depósito de cuentas anuales. Los enlaces que pones no dicen nada al respecto, y nunca he sabido de la obligatoriedad de esto. |
@pedrobaeza A qué te refieres con el último comentario? |
En el README haces referencias a unos enlaces sobre el depósito de cuentas anuales, y en esos documentos en ningún momento se menciona este informe con este formato, y obligatorio no es. Basta con un SyS de las cuentas sin agrupar. |
de no-autoexpansión de cuentas.
…om:Eficent/l10n-spain into 10.0-add-l10n_es_mis_reports_trial_balance
@pedrobaeza he actualizado el README. Aclarar que este módulo únicamente sirve para poder sacar el sumas y saldos con formato jerárquico, y que en v11 ya no va a ser necesario. |
@jbeficent no sería mejor hacer backport del modelo account.group + la ampliación de account_financial_report en v10 en un módulo account_financial_report_account_group? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unrelated travis error
Sigo pensando que la sobrecarga de datos que supone este módulo (sólo hay que ver el tiempo que tarda en actualizar cada vez) es mucho, y en que en su lugar se debería hacer backport de los grupos y la funcionalidad en account_financial_report. |
Estoy de acuerdo. Fue una prueba de concepto
El vie., 6 jul. 2018 19:23, Pedro M. Baeza <[email protected]>
escribió:
… Sigo pensando que la sobrecarga de datos que supone este módulo (sólo hay
que ver el tiempo que tarda en actualizar cada vez) es mucho, y en que en
su lugar se debería hacer backport de los grupos y la funcionalidad en
account_financial_report.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#789 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AHU_VsMYapbytL9G3EDbDYkcw2mvHo7Bks5uD50GgaJpZM4SZ8yP>
.
|
Plantillas MIS Builder para Informe de Sumas y Saldos Español
Incluye las siguientes plantillas para el motor de informes provisto
por el módulo mis_builder:
Las plantillas están basadas en los modelos para el depósito de cuentas anuales
del Registro Mercantil:
Instalación
Este módulo depende del módulo 'l10n_es_mis_reports' y del 'mis_builder' que
puede obtenerse en https://apps.odoo.com, o bien en https://github
.com/OCA/mis-builder.
Configuración
indicando los periodos deseados, y usando una de las plantillas
proporcionadas por este módulo.
Para cada informe hay una versión con sufijo 'Expandido' que incluye el
desglose de todas las subcuentas, más alla de la los niveles oficiales
definidos por el PGCE.
El informe sólamente considera las cuentas numeradas de acuerdo con el
formato establecido por el PGCE.