Fix the fix c3fefd2 :-( #155

Merged
merged 1 commit into from Dec 9, 2015

Conversation

Projects
None yet
3 participants
@guewen
Member

guewen commented Dec 3, 2015

The first item contains the name of the DB such as 'odoo_db' and the
second item contains 'user=gbaconnier dbname=odoo_db' which is what
expects psycopg2.connect.

I feel stupid about my first correction especially since I was sure to have
correctly tested it (automated test is hard here). At least, it does not
prevent jobs to run.

Fix the fix c3fefd2 :-(
The first item contains the name of the DB such as 'odoo_db' and the
second item contains 'user=gbaconnier dbname=odoo_db' which is what
expects psycopg2.connect.
@pedrobaeza

This comment has been minimized.

Show comment
Hide comment
@pedrobaeza

pedrobaeza Dec 3, 2015

Contributor

👍

Contributor

pedrobaeza commented Dec 3, 2015

👍

@lmignon

This comment has been minimized.

Show comment
Hide comment
@lmignon

lmignon Dec 3, 2015

Contributor

👍 If you feel stupid, I share your feeling since I've approved the previous fix 😏

Contributor

lmignon commented Dec 3, 2015

👍 If you feel stupid, I share your feeling since I've approved the previous fix 😏

lmignon added a commit that referenced this pull request Dec 9, 2015

@lmignon lmignon merged commit ef33ae1 into OCA:8.0 Dec 9, 2015

2 checks passed

ci/runbot runbot build 3136600-155-45ca64 (runtime 31s)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment