openerp-icm

Merge lp://qastaging/~agilebg/openerp-icm/7.0_fix_sale_commission into lp://qastaging/~openerp-icm/openerp-icm/7.0

  1. 7.0_fix_sale_commission
  2. Merge into 7.0
Proposed by Alex Comba - Agile BG
Status: Merged
Merged at revision: 16
Proposed branch: lp://qastaging/~agilebg/openerp-icm/7.0_fix_sale_commission
Merge into: lp://qastaging/~openerp-icm/openerp-icm/7.0
Diff against target: 12 lines (+1/-1)
1 file modified
sale_commission/settled.py (+1/-1)
To merge this branch: bzr merge lp://qastaging/~agilebg/openerp-icm/7.0_fix_sale_commission
Reviewer Review Type Date Requested Status
Yajo (community) Approve
Davide Corio (community) Approve
Lorenzo Battistini (community) Approve
Maxime Chambreuil (http://www.savoirfairelinux.com) Approve
Pedro Manuel Baeza code review Approve
Review via email: mp+205176@code.qastaging.launchpad.net

Description of the change

Fixing of the bug reported here: https://bugs.launchpad.net/openerp-icm/+bug/1277110

To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Of course, very basic fix. I suppose that nobody has used this module yet. I'm going to start checking it on two weeks for one customer.

Regards.

review: Approve (code review)
Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) :
review: Approve
Revision history for this message
Lorenzo Battistini (elbati) :
review: Approve
Revision history for this message
Davide Corio (enlightx-deactivatedaccount) wrote :

My bad, i submitted a wrong rev with the previous mp. but nobody noticed the issue ;-)

review: Approve
Revision history for this message
Yajo (yajo) wrote :

This allowed me to install the module.

review: Approve

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