Merge lp:~acsone-openerp/openerp-connector/7.0-connector-parameterized-job-desciption into lp:~openerp-connector-core-editors/openerp-connector/7.0
Status: | Merged |
---|---|
Merged at revision: | 598 |
Proposed branch: | lp:~acsone-openerp/openerp-connector/7.0-connector-parameterized-job-desciption |
Merge into: | lp:~openerp-connector-core-editors/openerp-connector/7.0 |
Diff against target: |
155 lines (+39/-8) 3 files modified
connector/CHANGES.rst (+3/-0) connector/queue/job.py (+16/-7) connector/tests/test_job.py (+20/-1) |
To merge this branch: | bzr merge lp:~acsone-openerp/openerp-connector/7.0-connector-parameterized-job-desciption |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stéphane Bidoul (Acsone) (community) | Approve | ||
Guewen Baconnier @ Camptocamp | code review | Approve | |
Review via email: mp+195572@code.launchpad.net |
Description of the change
Improve Job description
=======
Jobs are tasks to execute later. While the functionality is really robust and performs well, it is not always easy for the end user to visually distinguish the purpose of a job in the list of jobs. The description displayed in the list is computed from the documentation of the delayed function or from its name and there is no way to particularize this message by job instance.
This proposal aims to address this limitation by adding to the delay function a new optional parameter 'description'. The description given in the call to the delay function is used and displayed for the description of the job instead of the default one.
This proposal is backward compatible.
This is a very interesting improvement and well done!
Thanks