Merge lp://qastaging/~savoirfairelinux-openerp/openerp-canada/fix_l10n_ca_check_print_top into lp://qastaging/openerp-canada
| Status: | Merged |
|---|---|
| Approved by: | Sandy Carter (http://www.savoirfairelinux.com) |
| Approved revision: | 30 |
| Merged at revision: | 30 |
| Proposed branch: | lp://qastaging/~savoirfairelinux-openerp/openerp-canada/fix_l10n_ca_check_print_top |
| Merge into: | lp://qastaging/openerp-canada |
| Diff against target: |
930 lines (+474/-313) 7 files modified
l10n_ca_account_check_writing/__init__.py (+0/-1) l10n_ca_account_check_writing/__openerp__.py (+21/-6) l10n_ca_account_check_writing/account_voucher.py (+2/-2) l10n_ca_account_check_writing/report/README.txt (+9/-0) l10n_ca_account_check_writing/report/__init__.py (+0/-3) l10n_ca_account_check_writing/report/l10n_ca_check_print.py (+32/-27) l10n_ca_account_check_writing/report/l10n_ca_check_print_top.rml (+410/-274) |
| To merge this branch: | bzr merge lp://qastaging/~savoirfairelinux-openerp/openerp-canada/fix_l10n_ca_check_print_top |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Maxime Chambreuil (http://www.savoirfairelinux.com) | Approve | ||
| Joao Alfredo Gama Batista | code review, no tests | Approve | |
| Sandy Carter (http://www.savoirfairelinux.com) | code review | Approve | |
|
Review via email:
|
|||
To post a comment you must log in.
Revision history for this message
| Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter) wrote | # |
review:
Approve
(code review)
Revision history for this message
| Joao Alfredo Gama Batista (joao-gama) wrote | # |
review:
Approve
(code review, no tests)
Revision history for this message
| Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
LGTM