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

Debate: cambio de nombre del menú Declaraciones AEAT #4020

Open
acysos opened this issue Feb 17, 2025 · 11 comments
Open

Debate: cambio de nombre del menú Declaraciones AEAT #4020

acysos opened this issue Feb 17, 2025 · 11 comments
Assignees

Comments

@acysos
Copy link
Member

acysos commented Feb 17, 2025

Hola,

Nos estamos encontrando que empieza a existir modelos fiscales propios de ciertas zonas geográficas. Hasta ahora teníamos el menú de Declaraciones AEAT donde se engloba todo, mi propuesta es sencilla es cambiarle el nombre a Declaraciones o Declaraciones Impuestos. No poner AEAT, sino estamos creando un menú nuevo para cada zona incrementan los menú.

Por ejemplo, Declaraciones ATC, podrían ir el 415 y 420 englobados dentro de Declaraciones, pasaría lo mismo con el modelo 140 del Batuz del país vasco.

Qué opináis le cambiamos el nombre o uno más genérico y dejamos un solo menú principal, que luego tendrá los que requiera cada zona?

Saludos,
Ignacio

@acysos acysos self-assigned this Feb 17, 2025
@pedrobaeza
Copy link
Member

Bueno, con las de ATC creamos un nuevo menú de segundo nivel "Declaraciones ATC": https://github.com/OCA/l10n-spain/blob/16.0/l10n_es_atc/views/atc_menuitem.xml#L4, pero podemos reformularlo a tener un único menú superior y poner tres niveles:

  • Declaraciones

    • AEAT

      • Modelo 303
    • ATC

      • Modelo 415

@acysos
Copy link
Member Author

acysos commented Feb 17, 2025

@pedrobaeza me parece también bien, la idea es no llenar de menús el principal que puede ser al final muy poco visual.

Pero también hay que tener en cuenta que el por ejemplo el 347 se llama igual en varios sitios, pero hay alguna diferencia por ejemplo en Guipuzcoa se exporta diferente. No sería AEAT, aunque realmente el cálculo es el mismo, especificamente, pero si tiene una diferencia. Tal vez un así, aunque aquí no lo tengo tan claro, porque veo que muchos van a ser generales:

  • Declaraciones

    • Generales
      • Modelo 347
    • AEAT
      • Modelo 303
    • ATC
      • Modelo 415

Saludos,
Ignacio

@pedrobaeza
Copy link
Member

Uf, eso no me termina de convencer, porque en Canarias creo que no existe 347, por lo que tenerlo en el apartado "Generales" se puede entender que también les afecta, siguiendo esa misma lógica.

@acysos
Copy link
Member Author

acysos commented Feb 17, 2025

Ok, pues lo dejamos como AEAT los comunes, aunque tengan formato diferente de exportación, como es el caso de Guipuzcoa.

  • Declaraciones

    • AEAT

      • Modelo 303
      • Modelo 347
    • ATC

      • Modelo 415
    • Bizkaia/Vizcaya

      • Modelo 140

....

@HaraldPanten
Copy link
Contributor

Ok, pues lo dejamos como AEAT los comunes, aunque tengan formato diferente de exportación, como es el caso de Guipuzcoa.

  • Declaraciones

    • AEAT

      • Modelo 303
      • Modelo 347
    • ATC

      • Modelo 415
    • Bizkaia/Vizcaya

      • Modelo 140

....

Me parece una buena idea 👍🏻

@pedrobaeza
Copy link
Member

OK, pues podemos hacer el cambio de 17 en adelante, ¿o queréis ir más atrás?

@acysos
Copy link
Member Author

acysos commented Feb 17, 2025

Yo haría 16.0, 17.0 y 18.0 (cuando se migre).

Saludos,
Ignacio

@pedrobaeza
Copy link
Member

OK, pues adelante quien se anime a hacerlo.

@Christian-RB
Copy link
Contributor

Lo veo bien también, así sobrecargamos menos el menú. Lo único que no estoy seguro si deberíamos sacar el menu item fuera de l10n_es_aeat, en el caso de la ATC no nos afecta porque heredamos directamente de l10n_es_aeat pero el motivo principal es heredar de las plantillas de mapeo y los test helpers. Toda esa funcionalidad podría ir en un módulo común a parte.

@pedrobaeza
Copy link
Member

Yo creo que debe quedarse en l10n_es_aeat por el momento, ya que es una base adecuada para las declaraciones (sean de la Administración que sean) y no creo que surja el que algún modelo no dependa de este módulo y "vaya por libre" sin heredar el modelo base de declaración.

@acysos
Copy link
Member Author

acysos commented Feb 18, 2025

Asigno yo a un programador y que mande el PR

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

No branches or pull requests

4 participants