-
Notifications
You must be signed in to change notification settings - Fork 411
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Facebook Count #296
base: master
Are you sure you want to change the base?
Fix Facebook Count #296
Conversation
@@ -24,7 +24,7 @@ SharrrePlatform.register("facebook", function (options) { | |||
return { | |||
settings: defaultSettings, | |||
url: function (url) { | |||
return "https://graph.facebook.com/fql?q=SELECT%20url,%20normalized_url,%20share_count,%20like_count,%20comment_count,%20total_count,commentsbox_count,%20comments_fbid,%20click_count%20FROM%20link_stat%20WHERE%20url=%27{url}%27&callback=?"; | |||
return "http://graph.facebook.com/?id={url}&callback=?"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be https...
Cheers mate |
Still showing up issues on my end after replacing the minified file in production. |
@aandrewjoyce any errors in console? What happens in the network tab? What's the production URL? (It's a little hard to debug/help without any more info). |
@aandrewjoyce have you managed to get any share counts using the graph api explorer? |
on my side, the call was done correctly to facebook, but the info was not displayed. |
Applying fix from #294 by @sphism