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

8.0 sale prodlot mrp #176

Closed
wants to merge 16 commits into from
Closed

8.0 sale prodlot mrp #176

wants to merge 16 commits into from

Conversation

Viggor
Copy link

@Viggor Viggor commented Jul 2, 2015

[WIP] Add and transmit lot in mrp from sale. Work with #177

@Viggor Viggor mentioned this pull request Jul 2, 2015
@Viggor
Copy link
Author

Viggor commented Jul 2, 2015

Will be adapted to work with #140

@pedrobaeza
Copy link
Member

I think this one is not covered, but you can make it a new module depending on the other one with this functionality

@Viggor
Copy link
Author

Viggor commented Jul 2, 2015

Yes i will review #140 and adapt my module to it

@pedrobaeza
Copy link
Member

Thanks for collaborating!

lot_id = fields.Many2one('stock.production.lot', 'Lot')

@api.multi
def _action_compute_lines(self, properties=None):
Copy link
Member

Choose a reason for hiding this comment

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

Please add a comment in the code, something like
We passe the production.id in the context so you can easily get retrieve the production information during the process that generate the line of needed for the production order. For exemple, you can in the function bom_explod read the information of the production order to customise the needed

Copy link
Author

Choose a reason for hiding this comment

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

Done!

@pedrobaeza
Copy link
Member

This is still red, so I'm closing. Feel free to reopen if still needed.

@pedrobaeza pedrobaeza closed this May 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants