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

[11.0][MIG] quality_control_stock #307

Merged
merged 19 commits into from Dec 28, 2018

Conversation

NachoAlesLopez
Copy link
Contributor

pedrobaeza and others added 14 commits September 11, 2018 12:23
…README files, and new concepts. WIP of quality_control_tolerance

Merge branch '8.0' of github.com:odoomrp/odoomrp-wip into 8.0

[FIX] quality_control_stock:
* File missing
* Triggers for product category
* Omit lot when it's not present
* Change icon set
* Added "partner_selectable" field, so that inspections can by made by partner
* Stock inspections are "partner_selectable"
* Little view fixing, field was oe_edit_only instead of label
* Added partner to creation from stock_picking
* Added tests
* Picking pack operations automatically created
* more tests to increase coveralls

[IMP] quality_control_stock:
* added demo data
* replaced unlink method by ondelete="cascade"
* required changes
Making it a regular field, you can put any manual quantity in inspections
quality_control_mrp: tests extended

[IMP] quality_control_mrp: removed not required code
@NachoAlesLopez NachoAlesLopez mentioned this pull request Sep 13, 2018
25 tasks
@pedrobaeza pedrobaeza added this to the 11.0 milestone Sep 13, 2018
@pedrobaeza
Copy link
Member

Please check CIs

create quality control inspection without necessarily being
quality control user. It is common that the person doing the
picking is not going to be the same as the one doing the inspection.

So when a stock user create the transfer, the system will create
the necessary qc inspections, regardless if the user has permissions
to process them.
@JordiBForgeFlow
Copy link
Sponsor Member

See NachoAlesLopez#1

…urity

11.0 mig quality control stock security
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

LGTM

@LoisRForgeFlow LoisRForgeFlow merged commit 0fae9fe into OCA:11.0 Dec 28, 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.

None yet

9 participants