Skip to content
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

Support Facebook Comments Plugin. #925

Merged
merged 1 commit into from Jun 2, 2016
Merged

Conversation

@hydai
Copy link
Contributor

hydai commented Jun 1, 2016

In this commit:
1. Upgrade Facebook SDK from v2.5 to v2.6 (2016/04/12)
2. Support Facebook Comments Plugin

You could find the demonstration via this link.

And the following is the screenshot from my demonstration page:
2016-06-02 12 27 05

In this commit:
    1. Upgrade Facebook SDK from v2.5 to v2.6 (2016/04/12)
    2. Support Facebook Comments Plugin
@iissnan iissnan merged commit cd9429b into iissnan:master Jun 2, 2016
1 check passed
1 check passed
hound No violations found. Woof!
@iissnan
Copy link
Owner

iissnan commented Jun 2, 2016

Thanks. 👍

@iissnan iissnan added the Docs label Jun 2, 2016
@tomroy
Copy link

tomroy commented Jun 3, 2016

Thank you, this is exactly what I wanted.

@iissnan iissnan added this to the v5.1.0 milestone Oct 19, 2016
@iissnan iissnan removed the Docs label Feb 27, 2017
habren pushed a commit to habren/hexo-theme-next that referenced this pull request Apr 8, 2017
@lavicle
Copy link

lavicle commented Jul 7, 2017

Thank you for your work.
However, the comment widget doesn't show on my site.
Instead, it shows "href should represent a valid URL" in a flash and then disappear where the comment widget should be.
Plus, it seems that facebook sdk works fine for me since the thumb can be displayed.
Thank you! @hydai

Problem solved! After correcting the href, the comment widget shows.

@hydai hydai deleted the hydai:fb-comments-plugin branch Sep 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.