search

OCA / product-kitting

Odoo Kitting Addons
GNU Affero General Public License v3.0
17 stars 45 forks source link

7.0 bom stock not shown #1

Closed arthru closed 9 years ago

arthru commented 10 years ago

This pull request propose to hide the field bom_stock if the product does not have any boms...

rvalyi commented 10 years ago

:+1: LGTM

pedrobaeza commented 10 years ago

Can you please fix the rest of the flake8 errors of the module:

./bom_stock/__openerp__.py:26:6: E131 continuation line unaligned for hanging indent
./bom_stock/__openerp__.py:27:80: E501 line too long (130 > 79 characters)
./bom_stock/__openerp__.py:32:39: E203 whitespace before ','

The rest LGTM.

arthru commented 10 years ago

@pedrobaeza here is the end of the pep8 cleanup.

pedrobaeza commented 10 years ago

:+1:

arthru commented 9 years ago

Can someone merge that ? Please note that travis fails due to the rest of the repo, not that module...

arthru commented 9 years ago

Thanks @pedrobaeza