Merge lp:~therp-nl/account-analytic/6.1-account_analytic_required_i18n_nl into lp:account-analytic/6.1
Proposed by
Stefan Rijnhart (Opener)
Status: | Merged |
---|---|
Merged at revision: | 13 |
Proposed branch: | lp:~therp-nl/account-analytic/6.1-account_analytic_required_i18n_nl |
Merge into: | lp:account-analytic/6.1 |
Diff against target: |
138 lines (+105/-4) 3 files modified
account_analytic_required/account.py (+14/-2) account_analytic_required/i18n/account_analytic_required.pot (+13/-2) account_analytic_required/i18n/nl.po (+78/-0) |
To merge this branch: | bzr merge lp:~therp-nl/account-analytic/6.1-account_analytic_required_i18n_nl |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Guewen Baconnier @ Camptocamp | code review, no test | Approve | |
Review via email: mp+159727@code.launchpad.net |
Description of the change
The most important translation string of the module, containing the error message shown to the user if a required analytic account is missing, does not occur in generated po files for the module. The reason is that the substitution arguments are included in the _() call. This branch moves the arguments out of the call, updates the pot file and adds Dutch translations for this very useful module.
To post a comment you must log in.
LGTM