Antiun / antiun-odoo-addons

Antiun Odoo public addons
17 stars 47 forks source link

[8.0] base_exports_manager - Cambio de nombre y error el bugtracker #6

Closed rafaelbn closed 8 years ago

rafaelbn commented 9 years ago

@antespi

base_exports_manager 8.0

@pedrobaeza recomienda cambiar el nombre a base_export_manager (sin s en export)

Además al querer abrir un issue con el bugtracker no me lo crea en nuestro repo, me lleva aquí (debe de estar mal la plantilla o habéis hecho un PR):

https://github.com/OCA/server-tools/issues/new

pedrobaeza commented 9 years ago

No, recomiendo cambiarlo a _base_exportextension (sin s y cambiando manager por extension).

rafaelbn commented 9 years ago

@pedrobaeza va a ser un Gestor no una Extensión. A priori será un gestor de listas de exportación para el admin, pero puede ser planteado para los usuarios también.

pedrobaeza commented 9 years ago

Lo que habeis hecho la convierte en una extension (mostrar ese campo). Otra extensión será ese gestor que comentas.

rafaelbn commented 9 years ago

@pedrobaeza el campo sólo se muestras si estás en modo desarrollador.

pedrobaeza commented 9 years ago

Bueno, pero es una extensión. Yo lo dejaría con ese nombre. Luego, haced lo que queráis. Si acaso, hay que ponerlo apellido: _base_export_loquehagamanager, pero desde luego, no lo otro, y el plural hay que tratar de evitarlo siempre.

rafaelbn commented 9 years ago

@antespi toma nota

rafaelbn commented 9 years ago

cc @antespi

rafaelbn commented 8 years ago

@JavierIniesta ¿Cómo está esto? podemos cerrar issue?

JavierIniesta commented 8 years ago

Es la primera noticia que tengo al respecto. De hecho, el PR a OCA/server tools está hecho con el nombre base_exports_manager

rafaelbn commented 8 years ago

PR en OCA: https://github.com/OCA/server-tools/pull/289 (siempre debemos referenciar)

rafaelbn commented 8 years ago

Ya está el módulo en OCA