Merge lp://staging/~therp-nl/therp-addons/7.0-partner_name_dutch_salutation into lp://staging/~therp-nl/therp-addons/7.0

Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 102
Proposed branch: lp://staging/~therp-nl/therp-addons/7.0-partner_name_dutch_salutation
Merge into: lp://staging/~therp-nl/therp-addons/7.0
Diff against target: 170 lines (+118/-0)
6 files modified
partner_name_dutch/__openerp__.py (+4/-0)
partner_name_dutch/data/res_partner_title.xml (+23/-0)
partner_name_dutch/i18n/nl.po (+34/-0)
partner_name_dutch/model/__init__.py (+1/-0)
partner_name_dutch/model/res_partner_title.py (+29/-0)
partner_name_dutch/view/res_partner_title.xml (+27/-0)
To merge this branch: bzr merge lp://staging/~therp-nl/therp-addons/7.0-partner_name_dutch_salutation
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) (community) Approve
Review via email: mp+203685@code.staging.launchpad.net
To post a comment you must log in.
103. By Holger Brunn (Therp)

[ADD] mention latest changes in manifest

104. By Holger Brunn (Therp)

[FIX] copy&paste mistake

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks, sorry I've missed this.

Salutations in English are capitalized (as in "Dear Madam" / "My dear Professor"). Can you change this in the data lines and update the translations?

review: Needs Fixing
105. By Holger Brunn (Therp)

[FIX] English salutations are capitalized

Revision history for this message
Holger Brunn (Therp) (hbrunn) wrote :

Thanks!

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks for the updates

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: