Skip to content

[18.0][MIG] quality_control_mrp_oca: Migration to 18.0#1736

Open
alfredoavanzosc wants to merge 28 commits intoOCA:18.0from
alfredoavanzosc:18.0-mig-quality_control_mrp_oca
Open

[18.0][MIG] quality_control_mrp_oca: Migration to 18.0#1736
alfredoavanzosc wants to merge 28 commits intoOCA:18.0from
alfredoavanzosc:18.0-mig-quality_control_mrp_oca

Conversation

@alfredoavanzosc
Copy link
Copy Markdown
Contributor

@alfredoavanzosc alfredoavanzosc commented Feb 23, 2026

No description provided.

@alfredoavanzosc alfredoavanzosc force-pushed the 18.0-mig-quality_control_mrp_oca branch 2 times, most recently from 5b14623 to 75fa9b6 Compare February 23, 2026 09:50
@alfredoavanzosc
Copy link
Copy Markdown
Contributor Author

alfredoavanzosc commented Feb 23, 2026

@remi-filament ... review please.

remi-filament
remi-filament approved these changes Feb 23, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@remi-filament remi-filament left a comment

Choose a reason for hiding this comment

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

Thanks @alfredoavanzosc
LGTM, Code review

@remi-filament remi-filament mentioned this pull request Mar 9, 2026
Open
sergiocorato
sergiocorato reviewed Mar 31, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@sergiocorato sergiocorato left a comment

Choose a reason for hiding this comment

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

The commit from pre-commit is missing, you should split the current one in two.

@alfredoavanzosc alfredoavanzosc force-pushed the 18.0-mig-quality_control_mrp_oca branch from 75fa9b6 to 42f47bc Compare April 1, 2026 07:00
@alfredoavanzosc
Copy link
Copy Markdown
Contributor Author

alfredoavanzosc commented Apr 1, 2026

The commit from pre-commit is missing, you should split the current one in two.

Thank you very much @sergiocorato , I have already done what you asked.

@sergiocorato
Copy link
Copy Markdown
Contributor

sergiocorato commented Apr 1, 2026

The commit from pre-commit is missing, you should split the current one in two.

Thank you very much @sergiocorato , I have already done what you asked.

Could you explain please? I see in the migration commit changes made from pre-commit.

pedrobaeza and others added 21 commits April 1, 2026 12:14
@pedrobaeza @alfredoavanzosc
[IMP] Total refactorization of quality control modules with new API, …
eca6b2c 
…README files, and new concepts. WIP of quality_control_tolerance

[FIX] quality_control: Triggers for product category
@oihane @alfredoavanzosc
[FIX] Little changes in quality modules
27810f8
@pedrobaeza @alfredoavanzosc
[FIX] quality_control_mrp: Some problems
53f175b 
[IMP] quality_control_mrp: Smartbutton to inspections
@oihane @alfredoavanzosc
[MOD] Fixed MO view
1572354
@pedrobaeza @alfredoavanzosc
[IMP] quality_control*: Change icon set
4e7886f
@oihane @alfredoavanzosc
[MOD] Trigger lines instead of test
ba5fb70
@pedrobaeza @alfredoavanzosc
[IMP] Expand AUTHORS
0e6baa8
@elicoidal @alfredoavanzosc
[IMP] Improve some README files
0085e9c
@oihane @alfredoavanzosc
[MIG] quality_control_mrp: migrated from odoomrp/odoomrp-wip
153f79a 
[IMP] quality_control_mrp: required changes
* Limit access by groups in views
* Use api.model
@pedrobaeza @alfredoavanzosc
[FIX] quality_control_mrp: api.model method. Fixes OCA#67
af7c2bb
@pedrobaeza @alfredoavanzosc
[IMP] quality_control_mrp: Add reference to mrp.production
07a25e7
@maurochip @alfredoavanzosc
[8.0][FIX] quality_control_mrp: ignore cancelled moves when creating …
71bdf55 
…inspections

When an inspection was created, it wasn't ignoring the cancelled moves.
@oihane @alfredoavanzosc
[IMP] quality_control_stock: if lot in reference, get its info
2cb9551 
quality_control_mrp: tests extended

[IMP] quality_control_mrp: take product from manufacturing order

[IMP] quality_control_mrp: removed not required code

[FIX] quality_control_mrp: expected singleton error

[FIX] quality_control_mrp: partial fabrications where creating more inspections than required
@sbidoul @alfredoavanzosc
[FIX] remove en.po that was erroneously created by transbot
80bd83e
@agaldona @alfredoavanzosc
[IMP] quality_control_stock, quality_control_mrp: set correct qty to …
8947012 
…tests
@SimoRubi @alfredoavanzosc
[MIG] quality_control_mrp: Migration to 10.0
d781aa6
@NachoAlesLopez @alfredoavanzosc
[MIG] quality_control_mrp:
04ae961 
Migration to 12.0
@CarlosRoca13 @alfredoavanzosc
[IMP] quality_control_mrp: black, isort
1829f07
@CarlosRoca13 @alfredoavanzosc
[MIG] quality_control_mrp_oca: Migration to v13.0
409abd5
@alfredoavanzosc
[MIG] quality_control_mrp_oca: Migration to 14.0
5dfffc7
@oihane @alfredoavanzosc
[IMP] quality_control_mrp_oca: pre-commit stuff
1c85d73
@alfredoavanzosc alfredoavanzosc force-pushed the 18.0-mig-quality_control_mrp_oca branch from 42f47bc to de526a5 Compare April 1, 2026 10:22
@alfredoavanzosc alfredoavanzosc force-pushed the 18.0-mig-quality_control_mrp_oca branch 2 times, most recently from 8e5cf6d to 1c998cf Compare April 1, 2026 10:31
@alfredoavanzosc
Copy link
Copy Markdown
Contributor Author

alfredoavanzosc commented Apr 1, 2026

@sergiocorato review please.

sergiocorato
sergiocorato approved these changes Apr 1, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@sergiocorato sergiocorato left a comment

Choose a reason for hiding this comment

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

Maybe pre-commit did nothing before migration?
Anyway it's good for me.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented Apr 1, 2026

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

clementmbr

This comment was marked as outdated.

Sign in to view

clementmbr
clementmbr suggested changes Apr 1, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@clementmbr clementmbr left a comment

Choose a reason for hiding this comment

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

You just need this small fix @alfredoavanzosc

quality_control_mrp_oca/views/mrp_production_view.xml Outdated
<record id="action_qc_inspection_per_production" model="ir.actions.act_window">
<field name="name">Quality inspections from production order</field>
<field name="res_model">qc.inspection</field>
<field name="view_mode">tree,form</field>
Copy link
Copy Markdown
Member

@clementmbr clementmbr Apr 1, 2026

Choose a reason for hiding this comment

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

Suggested change
<field name="view_mode">tree,form</field>
<field name="view_mode">list,form</field>

Copy link
Copy Markdown
Contributor Author

@alfredoavanzosc alfredoavanzosc Apr 7, 2026

Choose a reason for hiding this comment

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

Correction made. Thank you very much.

@alfredoavanzosc alfredoavanzosc force-pushed the 18.0-mig-quality_control_mrp_oca branch from 1c998cf to 03546b4 Compare April 7, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@clementmbr clementmbr clementmbr requested changes

@sergiocorato sergiocorato sergiocorato approved these changes

@remi-filament remi-filament remi-filament approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.