Merge lp:~tlnd/financisto/csv-export into lp:~financisto-dev/financisto/trunk
Proposed by
tlnd
Status: | Merged | ||||||||
---|---|---|---|---|---|---|---|---|---|
Approved by: | Denis Solonenko | ||||||||
Approved revision: | 60 | ||||||||
Merged at revision: | 82 | ||||||||
Proposed branch: | lp:~tlnd/financisto/csv-export | ||||||||
Merge into: | lp:~financisto-dev/financisto/trunk | ||||||||
Diff against target: |
344 lines (+136/-40) 8 files modified
assets/credits.htm (+1/-1) res/layout/csv_export.xml (+38/-25) res/values-de/strings.xml (+5/-3) res/values/common.xml (+5/-0) res/values/strings.xml (+2/-0) src/ru/orangesoftware/financisto/activity/CsvExportActivity.java (+55/-2) src/ru/orangesoftware/financisto/activity/MainActivity.java (+12/-5) src/ru/orangesoftware/financisto/export/CSVExport.java (+18/-4) |
||||||||
To merge this branch: | bzr merge lp:~tlnd/financisto/csv-export | ||||||||
Related bugs: |
|
||||||||
Related blueprints: |
Allow custom separator for values in CSV export
(Undefined)
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Denis Solonenko | Approve | ||
Review via email: mp+35904@code.launchpad.net |
Commit message
CSV export: Custom value separator and persistent preferences
Description of the change
Hey,
I tackled the blueprint to "Allow custom separator for values in CSV export". Besides the comma, the user now can also select a semicolon or a tabulator. Additionally, he can choose whether the headers should be exported. Furthermore, the selection is persistent, thus the user does not have to reselect his preferences every time he does an export.
It would be great if somebody could review the code. If necessary, I'm willing to rework it to get it merged eventually.
Regards,
Timo
To post a comment you must log in.
Looks good, thank you! Please integrate into trunk.