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.

Sign up for GitHub

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

Jump to bottom

[MIG] l10n_es_aeat_sii_oca: Migration to 15.0 #1958

Merged
merged 203 commits into from
Dec 31, 2021
Merged

[MIG] l10n_es_aeat_sii_oca: Migration to 15.0 #1958

merged 203 commits into from
Dec 31, 2021

Conversation

ValentinVinagre
Copy link
Contributor

@ValentinVinagre ValentinVinagre commented Dec 23, 2021 •
edited
Loading

@ValentinVinagre ValentinVinagre changed the title [WIP][MIG] l10n_es_aeat_sii_oca: Migration to 15.0 [MIG] l10n_es_aeat_sii_oca: Migration to 15.0 Dec 23, 2021
@ValentinVinagre ValentinVinagre force-pushed the 15.0-mig-l10n_es_aeat_sii_oca branch 5 times, most recently from a8de323 to 32a2274 Compare December 28, 2021 08:41
@ValentinVinagre
Copy link
Contributor Author

@pedrobaeza sabes como arreglar el conflicto del requirements?

@pedrobaeza
Copy link
Member

Por qué no haces rebase y lo arreglas ahí?

@ValentinVinagre
Copy link
Contributor Author

Por qué no haces rebase y lo arreglas ahí?

en local no me da ese problema :/

@pedrobaeza
Copy link
Member

Eso será porque no tienes el local actualizado. Haz primer git fetch origin, siendo origin el remote para OCA.

@ValentinVinagre ValentinVinagre force-pushed the 15.0-mig-l10n_es_aeat_sii_oca branch 2 times, most recently from 2317445 to d1ed3ac Compare December 28, 2021 09:07
@ValentinVinagre
Copy link
Contributor Author

Ahora si, gracias pedro 🥇

@ValentinVinagre ValentinVinagre force-pushed the 15.0-mig-l10n_es_aeat_sii_oca branch 2 times, most recently from 132269d to cffcb2f Compare December 28, 2021 11:18
acysos and others added 14 commits December 28, 2021 12:26
@acysos @ValentinVinagre
l10n_es_aeat_sii: presentación inmediata de impuesto
31d1fa6
@rubencr7 @ValentinVinagre
l10n_es_aeat_sii: Carga directa del certificado
12e3ffe
@acysos @ValentinVinagre
l10n_es_aeat_sii: se añade mapeo de impuestos, traducciones y permisos
202de6b 
l10n_es_aeat_sii: añadidas facturas de bienes de inversión e intracomunitarias. Se traspasa la configuración del WSDL a parametros
@fuentes73 @ValentinVinagre
l10n_es_aeat_sii: Mejoras del certificado + otros
07cdbac 
* Ocultar la contraseña del certificado
* Plantilla de cuentas solo obligatorio si "activar SII" activado
* Habilitar un certificado por compañía
* Permisos de acceso
* Tener en cuenta las fechas de los certificados

�[IMP] l10n_es_aeat_sii: No utilizar los parámetros del sistema y buscar solo certificados activos
@RamonGuiuGou @ValentinVinagre
l10n_es_aeat_sii. Add connector
32c2a12
@acysos @ValentinVinagre
l10n_es_aeat_sii: error en la búsqueda del certificado + correción de…
d3d4690 
…scripciones en facturas de proveedor
@ismaelcj @ValentinVinagre
l10n_es_aeat_sii, refactoring
e1d7692
@fuentes73 @ValentinVinagre
l10n_es_aeat_sii: Envío de facturas rectificativas
733609c
@ismaelcj @ValentinVinagre
l10n_es_aeat_sii, tests
23988f6
@RamonGuiuGou @ValentinVinagre
l10n_es_aeat_sii: Filtros + agrupación:
5ea138f 
* Añadir filtros a las facturas para saber si el SII se ha enviado o no
* Corregir la agrupacion de los impuestos si en varias línias de factura hay el mismo impuestos
@fuentes73 @ValentinVinagre
l10n_es_aeat_sii: Trabajos readonly, limitar campos, poder re-encolar…
7a4c4fb 
… y retardo configurable
@acysos @ValentinVinagre
l10n_es_aeat_sii: copy fix, send button, method send, new filter and …
aa950bc 
…translations
@RamonGuiuGou @ValentinVinagre
l10n_es_aeat_sii: Añadir transformación multidivisa en los importes &…
5546e85 
…& añadir en la difinicion del producto la configuración de la causa exenta y para poderla enviar a la agencia
@oihane @ValentinVinagre
l10n_es_aeat_sii: update tests
ec61989
omar7r and others added 4 commits December 28, 2021 12:26
@omar7r @ValentinVinagre
[FIX] l10n_es_aeat_sii_oca: hook for obtaining the partner to send SII
a7d3101
@OCA-git-bot @ValentinVinagre
l10n_es_aeat_sii_oca 14.0.1.8.2
218c110
@omar7r @ValentinVinagre
[FIX] l10n_es_aeat_sii: Proper check of supplier invoice number
69e0586 
Field name has changed.
@pedrobaeza @ValentinVinagre
[OU-FIX] l10n_es_aeat_sii_oca: folder field required
a66370e 
We fill it with a wildcard text for complying with the requirement, but
its content is not so important, specially if the certificate is
already in use, as this one is only used for composing the new decrypted
certificates file path, not existing ones.
@ValentinVinagre
Copy link
Contributor Author

Ahora si, listo 👍🏻

@HaraldPanten
Copy link
Contributor

/ocabot migration l10n_es_aeat_sii_oca

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Dec 28, 2021
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 28, 2021
Open
48 tasks
@ValentinVinagre
Copy link
Contributor Author

@pedrobaeza venga que ya solo falta este 😸

@pedrobaeza
Copy link
Member

Debes poner todos los cambios por el linter en el commit de migración. El problema es que no has seguido el proceso de la guía de migración (https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-15.0), en el que se pone que hay que hacer el commit del pre-commit con --no-verify para que no te obligue a cumplir el resto de linters.

@ValentinVinagre
Copy link
Contributor Author

Debes poner todos los cambios por el linter en el commit de migración. El problema es que no has seguido el proceso de la guía de migración (https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-15.0), en el que se pone que hay que hacer el commit del pre-commit con --no-verify para que no te obligue a cumplir el resto de linters.

¿Te refieres a los cambios realizados en los archivos 'oca_dependencies.txt' y 'requirements.txt'?

@pedrobaeza
Copy link
Member

No, todos los demás que has hecho en el commit de pre-commit (cambio de versión, quitar los string, etc).

@ValentinVinagre
Copy link
Contributor Author

No, todos los demás que has hecho en el commit de pre-commit (cambio de versión, quitar los string, etc).

ostras.... creía que eso se hacía en el black, isort, prettier. Ahora lo arreglo 🤦🏻‍♂️

@pedrobaeza
Copy link
Member

Revisa de nuevo el procedimiento y verás que no es lo esperado. Ahí son solo los cambios automáticos, no los manuales.

@ValentinVinagre
Copy link
Contributor Author

Revisa de nuevo el procedimiento y verás que no es lo esperado. Ahí son solo los cambios automáticos, no los manuales.

Toda la razón del mundo. Diría que ahora ya está como debe.

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-1958-by-pedrobaeza-bump-nobump, awaiting test results.

HaraldPanten
HaraldPanten approved these changes Dec 31, 2021
View reviewed changes
Copy link
Contributor

@HaraldPanten HaraldPanten left a comment

Choose a reason for hiding this comment

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

Gracias Pedro

@OCA-git-bot OCA-git-bot merged commit a148d7d into OCA:15.0 Dec 31, 2021
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 97e9e7c. Thanks a lot for contributing to OCA. ❤️

@HaraldPanten HaraldPanten deleted the 15.0-mig-l10n_es_aeat_sii_oca branch December 31, 2021 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HaraldPanten HaraldPanten HaraldPanten approved these changes

@pedrobaeza pedrobaeza pedrobaeza approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
15.0
Development

Successfully merging this pull request may close these issues.