Merge lp://staging/~akretion-team/openerp-manufacturing/70-add-mrp-repair-extension into lp://staging/openerp-manufacturing

Proposed by Alexis de Lattre
Status: Needs review
Proposed branch: lp://staging/~akretion-team/openerp-manufacturing/70-add-mrp-repair-extension
Merge into: lp://staging/openerp-manufacturing
Diff against target: 516 lines (+490/-0)
5 files modified
mrp_repair_extension/__init__.py (+23/-0)
mrp_repair_extension/__openerp__.py (+43/-0)
mrp_repair_extension/i18n/mrp_repair_extension.pot (+129/-0)
mrp_repair_extension/mrp_repair.py (+59/-0)
mrp_repair_extension/mrp_repair_view.xml (+236/-0)
To merge this branch: bzr merge lp://staging/~akretion-team/openerp-manufacturing/70-add-mrp-repair-extension
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza code review Needs Fixing
Maxime Chambreuil (http://www.savoirfairelinux.com) Approve
Review via email: mp+211696@code.staging.launchpad.net

Description of the change

This MP adds the module mrp_repair_extension.

For those who have already played with the module mrp_repair from the official adds, they may have seen that it contains horrible bugs (such as https://bugs.launchpad.net/openobject-addons/+bug/1281687, which is now fixed) and that the usability is quite poor. The module mrp_repair_extension adds several small usability improvements to the mrp_repair module. It contains several small changes, in particular in the views. Each change is too small to be located in a standalone module, that's why I grouped them in this module. This module must not contain any big change to the mrp_repair module, because big changes should be in dedicated modules.

To post a comment you must log in.
Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) :
review: Approve
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Hi, Alexis,

Please explain on manifest description the modifications that this module makes.

Regards.

review: Needs Fixing (code review)

Unmerged revisions

7. By Alexis de Lattre

Add module mrp_repair_extension.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches