Merge lp://staging/~fabien-morin/unifield-server/fm-us-791-fixes into lp://staging/unifield-server
Proposed by
Fabien MORIN
| Status: | Rejected |
|---|---|
| Rejected by: | jftempo |
| Proposed branch: | lp://staging/~fabien-morin/unifield-server/fm-us-791-fixes |
| Merge into: | lp://staging/unifield-server |
| Diff against target: |
181 lines (+14/-45) 3 files modified
bin/addons/account/report/account_balance_sheet.py (+0/-17) bin/addons/account/report/account_general_ledger.py (+14/-18) bin/addons/account/report/account_profit_loss.py (+0/-10) |
| To merge this branch: | bzr merge lp://staging/~fabien-morin/unifield-server/fm-us-791-fixes |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| jftempo | Pending | ||
|
Review via email:
|
|||
Description of the change
hi, I find that some code was useless and should be removed.
And also a code part that was not applied on trunk, may be because we were working me and Vincent on the same thing, I don't know.
To post a comment you must log in.
Unmerged revisions
- 3684. By Fabien MORIN
-
US-791 [IMP] remove useless whitespaces
- 3683. By Fabien MORIN
-
US-791 [FIX] account 8 and 9 should be displayed in the general ledger report.
Anyway, the way to display or not account should not be hardcoded anymore,
there is a new checkbox for this purpose on account view. - 3682. By Fabien MORIN
-
US-791 [IMP] remove the commented code about not displaying account 8 and 9
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
