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

New module stock_reserve_mrp #102

Closed
wants to merge 8 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 17, 2015

This module allows manufacturing orders to reserve stock.

Stock Reserve MRP
=================

This module allows an automatic reserve when the product is finished.

Choose a reason for hiding this comment

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

Thanks for your contribution.
s/reserve/reservation

@oca-clabot
Copy link

Hey @dg-clearcorp, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla
Here is a list of the users:

  • @lg-clearcorp (login unknown in OCA database)

Appreciation of efforts,
OCA CLAbot

from openerp import models, fields, api


class mrp_bom(models.Model):
Copy link
Member

Choose a reason for hiding this comment

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

in stock.py you use camel case for the class name which is the recommended practise. Can you rename this class to MrpBom?

@gurneyalex
Copy link
Member

In addition to the notes above, could you:

  • move the stock.py and mrp.py files to a models/ subpackage
  • move the views to a views/ subdirectory
  • add an automated test

Thanks for your time

@gurneyalex
Copy link
Member

user lost interes, no CLA, old PR -> closing

@gurneyalex gurneyalex closed this Mar 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants