theonion / django-bulbs

DEPRECATED: This project is now part of the Mono Repo (https://github.com/theonion/omni)
MIT License
26 stars 7 forks source link

share tools require http protocol #280

Closed MelizzaP closed 8 years ago

MelizzaP commented 8 years ago

Fix for share tools, twitter requires http protocol

test links: http://test-link.test.theonion.com/series/onion-film-standard' http://test-link.test.theonion.com/video/scientists-continue-developing-alternative-energy--51011

@spra85 @collin

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 86.178% when pulling e48d0e2c54b7ee3a41014d43ca291fbc14ef434f on share-tools-fix into d87162c5bc901225952f0d9e4fa3cbcf33c3265c on master.

spra85 commented 8 years ago

👍

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 86.178% when pulling 423fe9a196f8eb749729d05e4be91db39ad7d031 on share-tools-fix into d87162c5bc901225952f0d9e4fa3cbcf33c3265c on master.

mparent61 commented 8 years ago

👍 As mentioned, prefer "https" over "http" if possible

mparent61 commented 8 years ago

Whoops nevermind my HTTPS comment