Merge lp://staging/~dreis-pt/therp-addons/7.0-port-users_ldap_populate-dr into lp://staging/~therp-nl/therp-addons/7.0

Hi Daniel,

thanks! I think this module should be moved to a community project now, maybe server-env-tools?

Pending that, I'd like to suggest for l.9 to please use at least two digit version numbers, to eventually be able to follow the convention of different levels for version or features, and one for minor changes.

Looks good otherwise. Please claim your credits with an additional "(C) 2013" and @author mention in the python files

Oh, I was not explicit enough in my review but I would like it if you could propose this module to a community project, replacing this proposal.

Hi Stefan,

It's good to know you agree to "donate" these modules to the Community.
I'll also port the `users_ldap_groups` and, if you agree, will propose them both for server-env-tools.

Regarding authorship, I think it's too much for me to claim for that when I'm only doing a small contribution.
Actually this is an interesting question for OCA: when should a contributor be added to the author list (if that still makes sense under OCA contribution agreement), and should contributors be given credit. But that's another discussion ...

Hi Daniel,

IMHO upgrading and testing any module under a new OpenERP major release is a substantial and valuable work that should be credited, but you may do as you see fit yourself, obviously ;-)

We are happy to hear that users_ldap_groups is useful to you too and looking forward to seeing that one proposed in a community project.

Rejecting this proposal because of follow up proposal on other project: https://code.launchpad.net/~dreis-pt/server-env-tools/7.0-port-users-ldap/+merge/176355

Stephan: these modules should be removed from your 7.0 branch then.
Or maybe leave them with only an inactive __openerp__.py file informing where the modules moved to.

Already taken care of ;-)

https://code.launchpad.net/~therp-nl/therp-addons/7.0-remove_ldap_modules/+merge/176371