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

[WIP][17.0][MIG] l10n_es_aeat_sii_oca: migration to 17.0 #3504

Closed

Conversation

manuelregidor
Copy link
Contributor

@manuelregidor manuelregidor commented Mar 28, 2024

Con la migración se llevará a cabo una refactorización para eliminar la dependencia con el módulo queue_job y realizar los envíos a través de triggers.

T-5833

@manuelregidor manuelregidor marked this pull request as draft March 28, 2024 11:13
@manuelregidor
Copy link
Contributor Author

@pedrobaeza ¿Hay algún cambio en este módulo que te gustaría realizar para incluirlo en la migración? Gracias

@pedrobaeza
Copy link
Member

Sí, lo comenté aquí: #3298 (comment). De hecho, lo iba a coger yo.

@HaraldPanten
Copy link
Contributor

@pedrobaeza te lo dejamos a ti, entonces? O miramos conjuntamente y lo llevamos adelante? Lo que prefieras.

@pedrobaeza
Copy link
Member

Voy yo a echar un vistazo y os digo.

@HaraldPanten
Copy link
Contributor

/ocabot migration l10n_es_aeat_sii_oca

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Apr 2, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 2, 2024
48 tasks
@HaraldPanten
Copy link
Contributor

Hola @pedrobaeza has podido avanzar en el nuevo planteamiento?

Saludos.

@pedrobaeza
Copy link
Member

De momento no.

@HaraldPanten
Copy link
Contributor

A tener en cuenta --> #3550

@HaraldPanten
Copy link
Contributor

A tener en cuenta --> #3572

@manuelregidor manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_sii_oca branch 2 times, most recently from 09d8152 to 19352b0 Compare June 3, 2024 08:32
@sistemastetrace
Copy link

hola a todos, esta previsto cuando puedes estar este modulo funcional? me encantaria aportar pero no tengo estos conocimientos.

@AitorAyala
Copy link

Hello, Can we know the forecast for when this module will be migrated to version 17? Thank you so much

@manuelregidor
Copy link
Contributor Author

@sistemastetrace @AitorAyala Estamos esperando a que la refactorización en v16 para VeriFACTU esté mergeada para así incluir directamente estos cambios en la migración (#3572)

@manuelregidor manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_sii_oca branch 5 times, most recently from 178cb81 to 45f43c2 Compare September 17, 2024 07:59
ozono and others added 6 commits September 17, 2024 10:03
… de los intracomunitarios. Para no presentar el 100% del padre.
- Se usa float-round en el redondeo
- También se redondean los valores en taxes_to
- Se incluye el redondeo de la BaseImponible
- Se aplica también a recibidas
…método "_get_sii_line_price_subtotal" definido para el cálculo de este dato.
pedrobaeza and others added 18 commits September 17, 2024 10:03
There's a new integrity check for avoiding inconsistent states. We were
using a trick on tests for quickly copying a customer invoice,
and converting the copy to vendor bill, but data got inconsistent doing
it.

For what we were testing, that was not a problem, but with the new
check, the operation is aborted.

Thus, we create the vendor bill from scratch, refactoring a method for
doing it easily.

We also take the occasion to improve a bit the number of invoice copies
done.
Starting from 14.0, root. prefix for channel name is no longer required

Fixes OCA#3147
Currently translated at 100.0% (201 of 201 strings)

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_aeat_sii_oca
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_aeat_sii_oca/es/
If you post more than one invoice at a time (using the action in the
list), and one of them is already sent to SII, there's a singleton error
because a typo in the code using `self` instead of `invoice`.
When sending a cancellation to SII, `FacturaExpedida` element in the
data structure is not built and sent, so current override (after the
mixin refactoring) is not taking into account this, crashing on
modifying a missing element.

We avoid the problem don't modifying such elements if cancelling.
In v13 this field was removed from SII result tab, while merging invoice
and vendor bill forms.
@manuelregidor manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_sii_oca branch from 45f43c2 to 3b8836e Compare September 17, 2024 08:03
@manuelregidor manuelregidor changed the title [WIP][17.0][MIG] l10n_es_aeat_sii_oca, migration to 17.0 [17.0][MIG] l10n_es_aeat_sii_oca: migration to 17.0 Sep 17, 2024
@manuelregidor manuelregidor marked this pull request as ready for review September 17, 2024 08:07
@manuelregidor manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_sii_oca branch from 3b8836e to 7a3b3a2 Compare September 17, 2024 10:43
@manuelregidor manuelregidor marked this pull request as draft September 17, 2024 10:52
@manuelregidor manuelregidor changed the title [17.0][MIG] l10n_es_aeat_sii_oca: migration to 17.0 [WIP][17.0][MIG] l10n_es_aeat_sii_oca: migration to 17.0 Sep 17, 2024
@manuelregidor manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_sii_oca branch from 7a3b3a2 to f452b3a Compare September 17, 2024 10:57
@manuelregidor manuelregidor force-pushed the 17.0-mig-l10n_es_aeat_sii_oca branch from f452b3a to b5808f4 Compare September 18, 2024 13:52
@manuelregidor manuelregidor deleted the 17.0-mig-l10n_es_aeat_sii_oca branch September 27, 2024 06:08
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.