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

Migration to version 11.0 #647

Closed
31 of 32 tasks
pedrobaeza opened this issue Oct 3, 2017 · 40 comments
Closed
31 of 32 tasks

Migration to version 11.0 #647

pedrobaeza opened this issue Oct 3, 2017 · 40 comments

Comments

@pedrobaeza
Copy link
Member

@pedrobaeza pedrobaeza commented Oct 3, 2017

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-11.0

Modules to migrate

@ddiazpinto
Copy link

@ddiazpinto ddiazpinto commented Oct 20, 2017

Hola!
Disculpad si estas cosas es tratan en otro hilo, pero he buscado y no he encontrado nada, a parte del de migración a la versión 10, que como es la anterior prefiero preguntar de nuevo aquí.

¿Con la nueva versión 11 de Odoo, cómo queda el ecosistema de módulos de este repositorio con respecto a la localización española? Veo que el módulo l10n_es ha sido integrado en el núcleo. Entiendo que va a mantenerlo esta misma comunidad y no va a tener los problemas de estancamiento que tenía el anterior, ¿es correcto? También veo que oficialmente están incluidos los modelos 111 y 303. ¿También son integraciones de estos módulos, o van por otro lado? ¿En qué estado están unos y otros? Por último, tengo previsto empezar a adecuar (en función de lo que me permita mi tiempo) los modelos 347 y 349. ¿Los que hay ahora son buena base para empezar? He visto que además hay alguna issue abierta por ahí.

Un saludo =)

@acysos
Copy link
Member

@acysos acysos commented Oct 20, 2017

El 111 y 303 de la Odoo oficial, son muy básicos, no tienen exportación a formato de la AEAT. Por lo tanto, se puede usar el core, pero hay que extenderlos.

Los anteriores siempre se pueden usar de base.

Saludos

@ddiazpinto
Copy link

@ddiazpinto ddiazpinto commented Oct 20, 2017

Gracias @acysos
¿Y es posible extenderlos o siguen tardando años en aceptar cada PR?

@emagdalenaC2i
Copy link

@emagdalenaC2i emagdalenaC2i commented Oct 21, 2017

Estoy probando los módulos ya migrados y me ha llamado la atención que las categorías y otros de los datos indicados en los archivos manifest.py no guardan la coherencia que creo que debían. Igual me equivoco, disculpad si es así pero:

  • "website" debería ser el mismo?
  • "category" debería seguir las categorías de los módulos principales y extra?
    En l10n_es_partner_mercantil está "category": "Localization/Europe"
    En l10n_es_account_asset/manifest.py está "category": "Accounting & Finance" y en esta categoría no existe ningún otro módulo principal o extra de Odoo

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Oct 21, 2017

@ddiazpinto el 111 y 303 que incorpora Odoo es solo explotable en enterprise, así que OCA/l10n-spain siempre proveerá una opción open-source de esto.

@emagdalenaC2i, sobre lo que comentas:

  • Del website, se acordó hace poco apuntarlo al repositorio, pero antes se dejaba libertad para poner lo que se quisiera, incluso la web del author. Como tampoco es crítico, la verdad es que no lo tenemos en cuenta en revisiones.
  • Algo parecido pasa con las categorías. Es un dato no muy tenido en cuenta, la verdad, y el propio Odoo ha ido cambiando de categorías a lo largo de las versiones. Con esa categoría, por ejemplo fue "Accounting", luego "Account & Finance" y ahora no sé cuál será... Si consideras que estaría mejor clasificarlo de otra manera, un PR cambiándolo siempre es bienvenido por supuesto.

@emagdalenaC2i
Copy link

@emagdalenaC2i emagdalenaC2i commented Oct 22, 2017

Ok, creado el PR.
No es que sea un cambio importante, pero mejor tratar de ordenar los módulos de la 11 ahora que se está empezando que más adelante. Además me permite hacer mi primera aportación para ver el proceso que seguís con un commit muy sencillo.

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Oct 22, 2017

Perfecto, pues echa un vistazo a https://odoo-community.org/page/Contribute para ver las formas de contribuir que hay y profundizar en la de código.

@emagdalenaC2i
Copy link

@emagdalenaC2i emagdalenaC2i commented Oct 22, 2017

Ya lo hice, aunque la verdad (como es habitual en este tipo de proyectos) me he encontrado partes de documentación sin actualizar y muchas lagunas.
Por ejemplo al asignar un nombre al PR y poner los prefijos [11.0] [FIX] [IMP] o similar. Si está definido en algún sitio, no lo he encontrado.
Con respecto a la traducción, también tengo dudas. Se realiza en Transifex? No está creado el Proyecto para la 11 y el de la 10 no tiene cadenas para traducir

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Oct 22, 2017

Efectivamente, lo de los prefijos no está regulado totalmente, aunque tienes una pequeña pauta en https://www.odoo.com/documentation/8.0/reference/guidelines.html#commit-message, pero no estaría mal incluirlo en las guidelines de OCA (no de Odoo), completándolo con los casos que no se dan en Odoo ([MIG] por ejemplo).

Sobre las traducciones, estamos migrando a la plataforma Weblate, y de momento se han desactivado la exportación a Transifex de las ramas 11.0 porque también había un problema con Python 3 y el script, aunque dicho problema se puede solucionar siguiendo una solución hecha por otro contribuidor. La pondré en marcha y se seguirá traduciendo en Transifex hasta nuevo aviso.

Y sí, también llevas razón en que los que estamos en la vorágine no nos preocupamos tanto de documentar las cosas que van cambiando para los nuevos que llegan, pero siempre se puede mejorar 😃

@kikopeiro
Copy link
Contributor

@kikopeiro kikopeiro commented Nov 13, 2017

Durante el Madrid Code Sprint me encargaré de migrar el modelo 115 (l10n_es_aeat_mod115)

@jsanchezNeuro
Copy link

@jsanchezNeuro jsanchezNeuro commented Nov 13, 2017

Buenas! Se sabe cuando se migrará el modulo payment_redsys ?

@agaldona
Copy link

@agaldona agaldona commented Nov 13, 2017

PR propuesto para la migración a la v11 del módulo l10n_es_dua. #686

@agaldona
Copy link

@agaldona agaldona commented Nov 14, 2017

PR propuesto para la migración a la v11 del módulo l10n_es_location_nuts. #691

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Nov 18, 2017

@jsanchezNeuro mira #695 para payment_redsys.

@gelo-landoo
Copy link
Contributor

@gelo-landoo gelo-landoo commented Feb 20, 2018

Perdón por la pregunta. El [l10n_es_vat_book] se portará a la V.11?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Feb 20, 2018

De momento no hay planes. Si os animáis vosotros o lo financiais, perfecto.

@jaumeplanas
Copy link
Contributor

@jaumeplanas jaumeplanas commented Apr 11, 2018

On l10n_es_account_banking_sepa_fsdd @jaumeplanas #818

@saecdata
Copy link

@saecdata saecdata commented Jun 6, 2018

Hola,
¿Por qué ya no es necesario el módulo l10n_es_account_fiscal_year_closing? ¿Las plantillas de cierre deben crearse a mano?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Jun 6, 2018

Porque directamente no hay el proceso de cierre contable. En las próximas jornadas de Odoo daré una charla al respecto: https://www.aeodoo.org/event/xi-jornadas-odoo-2018-2018-06-14-2018-06-15-7/page/introduccion-xi-jornadas-odoo-2018

@saecdata
Copy link

@saecdata saecdata commented Jun 6, 2018

¿Vas a hablar concretamente del cierre y la apertura de los ejercicios? Porque a nivel contable es necesario, si el proceso de cierre no es necesario cómo maneja Odoo el resultado del ejercicio (129)?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Jun 6, 2018

No, no es necesario, y sí, se hablará sobre cómo se hace ese proceso. Esta issue no es para tratar estos temas. Esto se ha hablado largo y tendido en las listas de la comunidad. Por favor busca dichos temas ahí.

@emagdalenaC2i
Copy link

@emagdalenaC2i emagdalenaC2i commented Jun 21, 2018

No es necesario migrar el l10n_es_aeat_mod130?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Jun 22, 2018

Gracias por el repaso que has dado, Eduardo. A ver, ese módulo solo es necesario cuando eres autónomo en régimen directo para la declaración del IRPF trimestral, y lo hice cuando yo estaba precisamente en ese régimen, pero es un caso raro que casi nadie lo tiene. No lo he seguido manteniendo puesto que ya no lo estoy, y además cambió el formato de exportación, por lo que había que rehacer esa parte.

@vipvalen
Copy link

@vipvalen vipvalen commented Apr 5, 2019

Hola.
Para cuando esta prevista la migración l10n_es_account_fiscal_year_closing de la v10 a v11?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Apr 5, 2019

Jeje, pues para nunca, porque no hace falta.

@vipvalen
Copy link

@vipvalen vipvalen commented Apr 6, 2019

Entonces como se debe de proceder para el cierre y el asiento de apertura, hay algún lugar donde se pueda ver como se debe de regularizar?

@vipvalen
Copy link

@vipvalen vipvalen commented Apr 6, 2019

Seria conveniente explicar en algún lugar de la localización el procedimiento a seguir, no?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Apr 6, 2019

Se explicó en las últimas jornadas españolas. Busca los vídeos.

@vipvalen
Copy link

@vipvalen vipvalen commented Apr 6, 2019

Puedes por favor enviarme los enlaces, te lo agradecería. Es complicado no saber donde esta una cosa tan importante documentada dentro de la localización española, en mi opinión.

Muchas gracias de antemano.

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Apr 6, 2019

@vipvalen
Copy link

@vipvalen vipvalen commented Apr 7, 2019

He visto el video y tengo una duda, entonces si ejecutas el módulo que hay en la v10 l10n_es_account_fiscal_year_closing duplicaría los datos en los informes?

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Apr 8, 2019

Impactaría en el saldo inicial y el debe/haber, sí, aunque el saldo final sea el mismo.

@dalonsod
Copy link
Contributor

@dalonsod dalonsod commented May 24, 2019

Hola, @pedrobaeza

Hemos visto que no están propuestos para migración l10n_es_payment_order ni l10n_es_payment_order_confirminet. Antes de que le echásemos un vistazo para migrar, ¿es por algún motivo en particular o realmente está disponible para ser migrado?

Gracias.

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented May 24, 2019

l10n_es_payment_order ya no es necesario porque ofrecía una infraestructura grande para unos formatos que ya no se utilizan como son los CSB 19, 58, etc. El de confirminet habría que refactorizarlo para no depender del anterior. De todas formas, no están en la lista porque la última versión de los mismos es la 8.0, y esta lista se obtiene automáticamente de lo que hay en la versión anterior. Aunque siempre se pueden añadir a mano los módulos que se consideren.

@dalonsod
Copy link
Contributor

@dalonsod dalonsod commented May 24, 2019

OK, el caso es que veo en la 10 esos dos addons, quizá se incorporaron después de haber creado la lista inicial de la 11. Pero entiendo que eso no cambia que, de migrar l10n_es_payment_order_confirminet debe de ser refactorizado como indicas.

Lo analizaremos para ver si nos ponemos con ello.

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented May 24, 2019

Eso es porque antes se utilizaba otra técnica para crear la nueva rama a partir de la anterior: se dejaba la rama como estaba, y se añadía un commit que pasaba todos los installable a False. Esto generaba bastante basura por módulos que nunca se llegan a migrar, como estos dos, o preguntas de "si está ahí el módulo, por qué no puedo instalarlo?", y desde la 11.0, se cambió el método para vaciar totalmente las ramas. La "desventaja" del nuevo método es tener que traer el historial de commits, pero está documentado en https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-11.0

@dalonsod
Copy link
Contributor

@dalonsod dalonsod commented May 24, 2019

Gracias, aclarado. No me había fijado en que estaban marcados con installable=False.

@pedrobaeza pedrobaeza pinned this issue Jun 10, 2019
@acysos
Copy link
Member

@acysos acysos commented Jul 29, 2019

Hola,

Porque no esta en la lista para migrar el módulo l10n_es_account_hierarchy_report? Es estético pero complicado de quitárselo a los contables.

Saludos

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Jul 29, 2019

@acysos ese módulo que yo sepa nunca ha estado aquí en la localización ni lo conocía... Ahora que lo veo vía apps, no creo que haga falta, ya que ya se incorporó a account_financial_report la posibilidad de sacar el SyS por níveles, que es un concepto muy parecido al de los dígitos.

@pedrobaeza
Copy link
Member Author

@pedrobaeza pedrobaeza commented Nov 6, 2019

Cierro también esta incidencia puesto que están todos los módulos importantes ya migrados.

@pedrobaeza pedrobaeza closed this Nov 6, 2019
@pedrobaeza pedrobaeza unpinned this issue Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet