[19.0][MIG] l10n_es_aeat_sii_oss: Migration to version 19.0 - #5010
Closed
flachica wants to merge 17 commits into
Closed
[19.0][MIG] l10n_es_aeat_sii_oss: Migration to version 19.0#5010flachica wants to merge 17 commits into
flachica wants to merge 17 commits into
Conversation
- Standard procedure - Conversion from account.invoice to account.move - Test adaptation TT31556
Steps to reproduce: - Have partner 1 with fiscal position FP1 with registration key 01. - Have partner 2 with fiscal position FP2 with registration key 02. - Create customer invoice with partner 1 as customer. - Select partner 2 as delivery address. Expected result: Fiscal position is changed to FP2 and the registration key is changed to 02. Current behavior: The fiscal position is changed, but not the registration key. That's because the recursive onchanges are explicitly disabled in account move object: https://github.com/odoo/odoo/blob/2f817a7b36cc7e5ab235829eecd61a1d71ce546e/addons/account/models/account_move.py#L1190 We workaround this limitation converting the registration key into a computed writable field that is computed even if the recursive onchange is disabled.
[ADD] módulo l10n_es_pos_sii para permitir enviar pedidos del TPV al sii Roadmap: - Comprobar que funcionalidad de account.move sigue siendo correcta. - Posibilidad de enviar al sii una sesión (tipo summary) - Envío devoluciones al SII -> POS - Cancelar envío al SII -> POS
Creación de un nuevo modelo en `l10n_es_aeat` llamado *aeat_mixin*, del que heredan *sii.mixin* (ya existente) y un nuevo *verifactu.mixin*. Refactor de *sii.mixin* pasando funcionalidad a *aeat_mixin* Se han renombrado varios campos sii_* a aeat_* ya que serán utilizados por ambos.
OCA-git-bot
added
series:19.0
mod:l10n_es_aeat_sii_oss
Contributor
Contributor
|
@flachica ya se ha fusionado el SII en V19. FYI. |
Contributor
Author
|
Gracias por el aviso @HaraldPanten |
Contributor
|
@flachica no era para que lo cerraras, sino para que pudierais seguir trabajando en este PR, ya que ya tenéis el módulo base disponible. Saludos! |
Contributor
Author
@HaraldPanten resulta que confundí la versión destino de mi cliente y no puedo seguir trabajando en este PR. Disculpa las molestias |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
10 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Standard migration