Merge lp:~camptocamp/oerpscenario/trunk-python-fix_account_chart_installation-nbi into lp:oerpscenario

Proposed by Nicolas Bessi - Camptocamp
Status: Merged
Merged at revision: 326
Proposed branch: lp:~camptocamp/oerpscenario/trunk-python-fix_account_chart_installation-nbi
Merge into: lp:oerpscenario
Diff against target: 13 lines (+2/-1)
1 file modified
features/steps/account_config.py (+2/-1)
To merge this branch: bzr merge lp:~camptocamp/oerpscenario/trunk-python-fix_account_chart_installation-nbi
Reviewer Review Type Date Requested Status
Romain Deheele - Camptocamp (community) code review, no test Approve
Alexandre Fayolle - camptocamp code review, no test Approve
Review via email: mp+192114@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

LGTM

review: Approve (code review, no test)
Revision history for this message
Romain Deheele - Camptocamp (romaindeheele) wrote :

LGTM too,

Romain

review: Approve (code review, no test)
Revision history for this message
Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount) wrote :

Thanks Then I will do the merge

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'features/steps/account_config.py'
2--- features/steps/account_config.py 2013-09-17 10:30:02 +0000
3+++ features/steps/account_config.py 2013-10-22 09:00:12 +0000
4@@ -59,7 +59,8 @@
5 assert company
6 configuration_wizard = model('wizard.multi.charts.accounts').create({'code_digits': digits,
7 'chart_template_id': template.id,
8- 'company_id': company.id})
9+ 'company_id': company.id,
10+ 'currency_id': company.currency_id.id})
11
12 vals = configuration_wizard.onchange_chart_template_id(template.id)
13 configuration_wizard.write(vals['value'])

Subscribers

People subscribed via source and target branches

to all changes: