bookieio / Bookie

Python based delicious.com replacement
GNU Affero General Public License v3.0
633 stars 138 forks source link

Adding BaseConnection and TwitterConnection db migration and model #466

Closed preddy5 closed 10 years ago

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-mysql/40/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-pgsql/101/

bookiebot commented 10 years ago

Test PASSed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-base/91/

sambuddhabasu commented 10 years ago

I have reviewed it :+1:

mitechie commented 10 years ago

Thanks for this update Prady, it's close but I think there's a few things to correct before landing. Let me know if you have any questions.

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-mysql/55/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-pgsql/117/

bookiebot commented 10 years ago

Test PASSed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-base/103/

mitechie commented 10 years ago

This looks almost ready. As part of the mysql work I've noticed that we should be more specific when creating constraints (like the foreign key ones) and the Unicode() references should be Unicode(255).

After that and a round of QA I think this will be ready to move forward and land.

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-mysql/56/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-pgsql/118/

bookiebot commented 10 years ago

Test PASSed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-base/104/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-mysql/57/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-pgsql/119/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-base/105/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-mysql/60/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-pgsql/122/

bookiebot commented 10 years ago

Test PASSed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-base/108/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-mysql/62/

bookiebot commented 10 years ago

Test FAILed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-pgsql/124/

bookiebot commented 10 years ago

Test PASSed. Refer to this link for build results: http://ci.bookie.io:8080/job/bookie-base/110/

mitechie commented 10 years ago

:+1: thanks for the updates

preddy5 commented 10 years ago

:shipit:

bookiebot commented 10 years ago

Status: merge request accepted. Url: http://ci.bookie.io:8080/job/bookie-lander