openerp-mexico-localization

Merge lp://staging/~vauxoo/openerp-mexico-localization/account_tax_category_dev_juan_carlos into lp://staging/openerp-mexico-localization

Proposed by Juan Carlos Hernandez on 2012-07-30

Status:	Merged
Merged at revision:	192
Proposed branch:	lp://staging/~vauxoo/openerp-mexico-localization/account_tax_category_dev_juan_carlos
Merge into:	lp://staging/openerp-mexico-localization
Diff against target:	311 lines (+274/-0) 7 files modified account_tax_category/__init__.py (+24/-0) account_tax_category/__openerp__.py (+47/-0) account_tax_category/account_tax_category.py (+52/-0) account_tax_category/account_tax_category_view.xml (+77/-0) account_tax_category/data/account_tax_category.xml (+57/-0) account_tax_category/security/account_tax_category_security.xml (+14/-0) account_tax_category/security/ir.model.access.csv (+3/-0)
To merge this branch:	bzr merge lp://staging/~vauxoo/openerp-mexico-localization/account_tax_category_dev_juan_carlos
Related bugs:	Link a bug report

Reviewer	Date Requested	Status
Isaac López Zúñiga	2012-07-30	Needs Fixing on 2012-08-03
Juan Carlos Hernandez (community)		Needs Resubmitting on 2012-08-02
Moisés López - http://www.vauxoo.com	2012-07-30	Pending
Review via email: mp+117353@code.staging.launchpad.net

add module account_tax_category

182. By Juan Carlos Hernandez on 2012-08-01: [IMP][account_tax_category]change name field acc_tax_tagetory_id to tax_tagetory_id
183. By Juan Carlos Hernandez on 2012-08-02: [IMP][account_tax_category] add company_id
184. By Juan Carlos Hernandez on 2012-08-02: [IMP][account_tax_category] add default y widget

Revision history for this message

Juan Carlos Hernandez (openerp1) wrote on 2012-08-02:

listo lo siguiente
El campo
acc_tax_tagetory_id
Corrígele el nombre please
tax_category_id
Por favor

Al modelo account.tax.category
agregale la _columns company_id
Y su default
Y su widget selection en la vista.

review: Needs Resubmitting

Revision history for this message

Isaac López Zúñiga (isaako34) wrote on 2012-08-03:

Carlos,

Podrías corregir el campo tax_tagetory_id por tax_category_id, como lo indica moy por favor.

También podrías agregar el campo tax_category_id con widget de tipo selection al modelo account_tax por favor.

Saludos.

review: Needs Fixing

Revision history for this message

Juan Carlos Hernandez (openerp1) wrote on 2012-08-03:

YA LO HICE SOLO ACTUALIZA EL BRANCH.

On Fri, Aug 3, 2012 at 2:13 PM, Isaac López Zúñiga - http://www.vauxoo.com <
<email address hidden>> wrote:

> Review: Needs Fixing
>
> Carlos,
>
> Podrías corregir el campo tax_tagetory_id por tax_category_id, como lo
> indica moy por favor.
>
> También podrías agregar el campo tax_category_id con widget de tipo
> selection al modelo account_tax por favor.
>
> Saludos.
> --
>
> https://code.launchpad.net/~vauxoo/openerp-mexico-localization/account_tax_category_dev_juan_carlos/+merge/117353
> You proposed
> lp:~vauxoo/openerp-mexico-localization/account_tax_category_dev_juan_carlos
> for merging.
>