Code review comment for lp:~acsone-openerp/openerp-connector/7.0-connector-parameterized-job-desciption

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

David Beal:
> Good idea,
>
> But perhaps we may use 'description' argument as complement of
> self.func.__doc__ or 'Function %s' and not instead of

Perhaps can you precise what you have in mind with some examples?
Appending them would give somewhat weird results (redundancy, unrelated messages).

We must keep things simple, this proposal stays simple while still allowing to fully customize the job descriptions. That's why a totally support it.

Laurent Mignon
> It's nice to contribute ;-)

Even more when the contribution is scrupulous about details (including tests, docs, ...) :-)

« Back to merge proposal