Merge lp://staging/~akretion-team/account-analytic/account-analytic-required-pep8 into lp://staging/~acsone-openerp/account-analytic/account_analytic_required-test_suite-sbi
Proposed by
Alexis de Lattre
| Status: | Merged |
|---|---|
| Merged at revision: | 19 |
| Proposed branch: | lp://staging/~akretion-team/account-analytic/account-analytic-required-pep8 |
| Merge into: | lp://staging/~acsone-openerp/account-analytic/account_analytic_required-test_suite-sbi |
| Diff against target: |
212 lines (+80/-31) 5 files modified
account_analytic_required/__init__.py (+1/-1) account_analytic_required/__openerp__.py (+10/-7) account_analytic_required/account.py (+46/-15) account_analytic_required/account_view.xml (+12/-1) account_analytic_required/tests/test_account_analytic_required.py (+11/-7) |
| To merge this branch: | bzr merge lp://staging/~akretion-team/account-analytic/account-analytic-required-pep8 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Raphaël Valyi - http://www.akretion.com (community) | Approve | ||
| Stéphane Bidoul (Acsone) | Pending | ||
|
Review via email:
|
|||
Description of the change
Flake8 valid.
Small code cleanup.
Add account analytic policy in tree view (make it much easier to check the configuration of all the account types).
To post a comment you must log in.
Revision history for this message
| Raphaël Valyi - http://www.akretion.com (rvalyi) wrote | # |
review:
Approve
Revision history for this message
| Stéphane Bidoul (Acsone) (sbi) wrote | # |
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
straightforward LGTM