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

[FIX] [16.0] l10n_es_aeat_mod303: Permitir rellenar las casillas 108 y 111 #3770

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

Shide
Copy link
Contributor

@Shide Shide commented Oct 15, 2024

Permitir que el usuario rellene las casillas 108 y 111 para las declaraciones posteriores a Septiembre del 2024.

MT-7092 @moduon @rafaelbn @pedrobaeza revisar si queréis :)

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@rafaelbn
Copy link
Member

Intentamos resolver #3703

@pedrobaeza pedrobaeza added this to the 16.0 milestone Oct 15, 2024
@Shide Shide force-pushed the 16.0-calculo_casillas_108_110-l10n_es_aeat_mod303 branch from c4c395a to efc6664 Compare October 16, 2024 09:46
@Shide Shide requested a review from yajo October 16, 2024 09:46
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.

La tag del commit debe ser [IMP], no [FIX].

name="casilla_108"
widget="monetary"
options="{'currency_field': 'currency_id'}"
attrs="{'invisible': [('statement_type', '!=', 'S')]}"
Copy link
Member

Choose a reason for hiding this comment

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

Estas casillas deben estar visibles si es una declaración normal?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@Shide Shide Oct 18, 2024

Choose a reason for hiding this comment

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

No, sólo en ciertas declaraciones. Ahora me da este error:

E030321
La casilla 108 no podrá tener contenido si no es una autoliquidación rectificativa con motivo 'Discrepancia criterio administrativo'

Lo retomo a partir del martes

Copy link
Member

Choose a reason for hiding this comment

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

Por eso decía, porque precisamente solo si es autoliquidación rectificativa se puede rellenar esas casillas. Este nuevo concepto, de momento está asemejado a "declaración complementaria", pero creo que no es lo mismo, y no sé si debiéramos ampliar los tipos de declaración. Por eso lo dejé sin rellenar, ya que solo para ese caso expreso deben utilizarse.

Pero insisto, @rafaelbn el error que tú tenías no era por esto, si no porque se seguía exportando con la configuración antigua, porque no lo cambiasteis en la declaración que se creó antes de actualizar esto.

Copy link
Member

Choose a reason for hiding this comment

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

Correcto todos @pedrobaeza @Shide . A partir del martes lo terminamos aclarando las dudas.

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.

5 participants