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

Añadidos informes MIS para entidades SFL #1074

Merged
merged 6 commits into from
May 8, 2019
Merged

Conversation

antoniocanovas
Copy link

Pymes "Sin Fin Lucrativo".

Balance y PyG para entidades Sin Fin Lucrativo
mis_report_balance_sme_sfl.xml
mis_report_pyg_sme_sfl.xml
@pedrobaeza pedrobaeza added this to the 12.0 milestone Apr 29, 2019
@pedrobaeza
Copy link
Member

Revisa estos errores de lint:

l10n_es_mis_report/__manifest__.py:26:5: W191 indentation contains tabs
l10n_es_mis_report/__manifest__.py:26:5: E101 indentation contains mixed spaces and tabs
l10n_es_mis_report/__manifest__.py:30:8: W191 indentation contains tabs
l10n_es_mis_report/__manifest__.py:30:8: E101 indentation contains mixed spaces and tabs

<odoo>
<data>
<record id="mis_report_es_balance_pymes_sfl" model="mis.report">
<field name="name">Balance PYMEs sin fin lucrativo (PGCE 2008)</field>
Copy link
Member

Choose a reason for hiding this comment

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

Para ser consistente con el nombre del plan contable, llámalo Balance entidades sin ánimo de lucro (PGCE 2008)

<odoo>
<data>
<record id="mis_report_es_pyg_pyme_sfl" model="mis.report">
<field name="name">Pérdidas y ganancias Pyme Sin Fin Lucrativo (PGCE 2008)</field>
Copy link
Member

Choose a reason for hiding this comment

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

Para ser consistente con el nombre del plan contable, llámalo Pérdidas y ganancias entidades sin ánimo de lucro (PGCE 2008)

Copy link
Author

Choose a reason for hiding this comment

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

Se puso este nombre porque el BOE distingue entre PYMESFL y otras grandes, lo dejo como tú creas que es mejor:
Captura de pantalla 2019-04-30 a las 23 00 19
Captura de pantalla 2019-04-30 a las 23 00 08

Copy link
Member

Choose a reason for hiding this comment

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

OK, ya veo, entonces no va mal encaminado lo que has puesto sin duda. Entonces te diría que pongas PYMESFL directamente para mantener una longitud adecuada. Luego hay un campo de descripción en el que se puede poner todo el nombre completo para aclarar a los que le queden dudas.

Copy link
Author

Choose a reason for hiding this comment

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

No entiendo el error que da el chequeo, hace referencia al 349.
Help !!

@pedrobaeza
Copy link
Member

Nada, por eso no te preocupes, Antonio. Es el índice de cobertura de tests, que aquí además no debería saltar, ya que no cambia con lo que había antes. Creo que salta porque no tienes tu rama hecho un rebase para tener lo último de la 12.0, pero eso, no es relevante.

Lo que sí te pediría y ya fusionamos, es poner el texto completo que me enseñaste en el campo de notas que te dije. ¿Podrías hacerlo o quieres que lo haga yo?

@antoniocanovas
Copy link
Author

antoniocanovas commented May 8, 2019 via email

@pedrobaeza
Copy link
Member

Era simplemente añadir el campo description. Lo puedes ver en el commit que he hecho: f0a8ddd

Fusiono con squash cuando termine Travis

@pedrobaeza pedrobaeza merged commit 715a1ff into OCA:12.0 May 8, 2019
pedrobaeza pushed a commit to Tecnativa/l10n-spain that referenced this pull request May 26, 2020
HaraldPanten pushed a commit to sygel-technology/l10n-spain that referenced this pull request Oct 16, 2020
pedrobaeza pushed a commit to sygel-technology/l10n-spain that referenced this pull request Oct 23, 2020
HaraldPanten pushed a commit to sygel-technology/l10n-spain that referenced this pull request Apr 7, 2021
misern2 pushed a commit to QubiQ/l10n-spain that referenced this pull request Jan 13, 2022
pedrobaeza pushed a commit to Tecnativa/l10n-spain that referenced this pull request Apr 4, 2023
mpascuall pushed a commit to APSL/l10n-spain that referenced this pull request May 6, 2024
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.

None yet

2 participants