Fix bug introduced in c97ebbd in jobrunner #152

Merged
merged 1 commit into from Dec 1, 2015

Conversation

Projects
None yet
4 participants
@guewen
Member

guewen commented Dec 1, 2015

Commit c97ebbd was a frontport from 7.0.
However, openerp.sql_db.dsn was changed in 8.0 in that it now returns a
tuple. Extract the db_name from the returned tuple.

Fix bug introduced in c97ebbd in jobrunner
Commit c97ebbd was a frontport from 7.0.
However, openerp.sql_db.dsn was changed in 8.0 in that it now returns a
tuple. Extract the db_name from the returned tuple.
@pedrobaeza

This comment has been minimized.

Show comment
Hide comment
@pedrobaeza

pedrobaeza Dec 1, 2015

Contributor

👍

Contributor

pedrobaeza commented Dec 1, 2015

👍

@lmignon

This comment has been minimized.

Show comment
Hide comment
@lmignon

lmignon Dec 1, 2015

Contributor

👍

Contributor

lmignon commented Dec 1, 2015

👍

@sbidoul

This comment has been minimized.

Show comment
Hide comment
@sbidoul

sbidoul Dec 1, 2015

Member

👍

Member

sbidoul commented Dec 1, 2015

👍

@pedrobaeza

This comment has been minimized.

Show comment
Hide comment
@pedrobaeza

pedrobaeza Dec 1, 2015

Contributor

Merging as the problem in documentation is unrelated.

Contributor

pedrobaeza commented Dec 1, 2015

Merging as the problem in documentation is unrelated.

pedrobaeza added a commit that referenced this pull request Dec 1, 2015

Merge pull request #152 from guewen/fix-jobrunner-job-pending
Fix bug introduced in c97ebbd in jobrunner

@pedrobaeza pedrobaeza merged commit c3fefd2 into OCA:8.0 Dec 1, 2015

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
ci/runbot runbot build 3136347-152-9bcf68
Details
@guewen

This comment has been minimized.

Show comment
Hide comment
@guewen

guewen Dec 1, 2015

Member

Thanks for the quick merge!
Docs are broken again 😭

Member

guewen commented Dec 1, 2015

Thanks for the quick merge!
Docs are broken again 😭

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment