OrchardCMS / OrchardDoc

Documentation for Orchard 1.x (not Orchard Core, for that see https://docs.orchardcore.net/).
http://docs.orchardproject.net
142 stars 242 forks source link

Add Disqus comments to Orchard Docs #91

Closed m8ker closed 9 years ago

m8ker commented 10 years ago

Added disqus comment snippet to _SiteLayout.cshtml and added a link to make a comment in the sidebar. Also added one line of css to give comments container some padding.

bleroy commented 10 years ago

Thanks for the contribution. Before we merge it however, I'd like this to be discussed at the weekly meeting: how should this appear? Is it going to be a distraction? Who is going to moderate the comments?

m8ker commented 10 years ago

My own two cents is that at the bottom of the page, it won't be a distraction. If you find you disagree, you could put it on a tab like Railscasts http://railscasts.com/episodes/417-foundation?view=comments does.

As for moderation, here is a page explaining Disqus' spam filtering https://help.disqus.com/customer/portal/articles/466244-dealing-with-spam and how different moderation options affect it.

If you decide you want to merge it, once you have pushed to the production site, you may need to log into disqus.com to get the website comments functioning. The “forum shortname” is “orcharddoc”. So the Disqus url is orcharddoc.disqus.com.

On Mon, Jun 23, 2014 at 8:54 PM, Bertrand Le Roy notifications@github.com wrote:

Thanks for the contribution. Before we merge it however, I'd like this to be discussed at the weekly meeting: how should this appear? Is it going to be a distraction? Who is going to moderate the comments?

— Reply to this email directly or view it on GitHub https://github.com/OrchardCMS/OrchardDoc/pull/91#issuecomment-46929504.

abhishekluv commented 10 years ago

Hello I would be interested to be a moderator for the comments.

bleroy commented 10 years ago

Thanks @abhishekluv .

m8ker: this is public, so you'll have to change this password now. Please only communicate it through private messages.

I'll make sure this gets discussed next week at the public meeting.

m8ker commented 10 years ago

sorry, will change

On Tue, Jun 24, 2014 at 2:03 PM, Bertrand Le Roy notifications@github.com wrote:

Thanks @abhishekluv https://github.com/abhishekluv .

m8ker: this is public, so you'll have to change this password now. Please only communicate it through private messages.

I'll make sure this gets discussed next week at the public meeting.

— Reply to this email directly or view it on GitHub https://github.com/OrchardCMS/OrchardDoc/pull/91#issuecomment-47030522.

m8ker commented 10 years ago

I have sent the new passwords to Sebastian's MS email address. Please let me know if I should send them to someone else.

On Tue, Jun 24, 2014 at 2:48 PM, Justin Haight haight@gmail.com wrote:

sorry, will change

On Tue, Jun 24, 2014 at 2:03 PM, Bertrand Le Roy <notifications@github.com

wrote:

Thanks @abhishekluv https://github.com/abhishekluv .

m8ker: this is public, so you'll have to change this password now. Please only communicate it through private messages.

I'll make sure this gets discussed next week at the public meeting.

— Reply to this email directly or view it on GitHub https://github.com/OrchardCMS/OrchardDoc/pull/91#issuecomment-47030522.

bleroy commented 10 years ago

Perfect, thanks.

agriffard commented 10 years ago

I also would like to be added as moderator.

bleroy commented 10 years ago

Did Sébastien share admin rights with anyone? What was his feedback on this?

agriffard commented 10 years ago

No and we haven't discuss about that during the previous weekly committee meetings

abhishekluv commented 10 years ago

We didn't discuss about this. Let's do it next meeting.

dcinzona commented 9 years ago

I don't think adding comments will help with clarity. Now that I'm clear on how to edit the docs (submitting PRs via github), I think what is currently set up works.

I wouldn't mind having a Slack team that is open to everyone specifically for Orchard...