artsy.github.io icon indicating copy to clipboard operation
artsy.github.io copied to clipboard

Comments: Adding comments

Open orta opened this issue 6 years ago • 21 comments

I've worked on a few large-scale OSS projects, and I believe that people find it easier to just leave a comment and rely on a contributor to explain a problem rather than consulting the documentation. I consider doing everything you can to make people find their own answers a strong part of defensive open source.

orta avatar Jul 15 '17 22:07 orta

This is an interesting system you've built to handle comments. Do you foresee any problems with using Github Issues for comments while also using Github Issues for issue tracking? Or do you think just having the label "Comment Thread" and using Github's filtering is sufficient for separating the two and keeping things organized?

Additionally, were there any surprises that came up when implementing this system? For example, needing to support some kind of content of a comment (maybe an attached file?) and styling it correctly that didn't cross your mind when first implementing this.

grantjbutler avatar Jul 17 '17 14:07 grantjbutler

Do you foresee any problems with using Github Issues for comments while also using Github Issues for issue tracking?

ATM, I don't think so - our issues in the artsy github blog are generally a "we should make this blog post sometime" and are really low traffic. I added a label + title prefix to differentiate between them, but I don't think it'll make it harder for any of the non-comment discussions. If it does become a problem, it is not hard to migrate blog post comments into it's own repo too.

Additionally, were there any surprises that came up when implementing this system?

Today, no, but after a few comments maybe. I re-used existing comment threads from inside the issue while figuring out how it should look and feel. So I had a good chunk of real-life data to work against.

That said, I hadn't even considered that using GH issues allows posting images ( and the occasional table) so I'm now considering those an extra freebie 🍡

orta avatar Jul 17 '17 16:07 orta

Having comments on GH issues also made it really easy to hook up to a slack channel - so that I can use my normal stream-based flow of issue management. 🎉

screen shot 2017-07-17 at 12 16 37

orta avatar Jul 17 '17 16:07 orta

Have you thought about displaying reactions at all?

M-Zuber avatar Jul 27 '17 07:07 M-Zuber

Yeah, I have an issue open on this repo to do it - https://github.com/artsy/artsy.github.io/issues/369

It looks pretty easy, would need to change the header sent off to the request to include a specific attribute. Probably an hour or two at some time for that + the css

orta avatar Jul 27 '17 11:07 orta

Fantastic tutorial, you really simplified the process especially with the gh-commentify app. Thanks!

mikeeus avatar Dec 11 '17 05:12 mikeeus

ATM, I don't think so

Half a year later, did you run into any problems?

mxstbr avatar Jan 15 '18 21:01 mxstbr

Nope, never had a single problem 👍

orta avatar Jan 16 '18 15:01 orta

Hi :wave: https://utteranc.es automates the blog-post to issue linking and allows users to post comments without navigating to GitHub :fire:

jdanyow avatar Apr 05 '18 04:04 jdanyow

Wow. This is amazing!

samselikoff avatar Jan 30 '19 17:01 samselikoff

How do you deal with XSS?

iamskok avatar Feb 14 '19 10:02 iamskok

GitHub's markdown parser handles that

orta avatar Feb 14 '19 12:02 orta

@jdanyow Lol about to say same thing

TheoFein avatar May 28 '19 15:05 TheoFein

This is cool

luckeysherpa07 avatar Nov 12 '19 06:11 luckeysherpa07

Cool idea.

orms-study avatar Feb 06 '20 04:02 orms-study

This looks cool. I am going to try this on my blog, but let me see first how does it go when I post :)

rehmatworks avatar Mar 30 '20 20:03 rehmatworks

Have you looked into the new GitHub feature for discussions?

breml avatar Dec 29 '20 16:12 breml

@breml No we haven't – what we have is still working for us. GitHub discussions seem interesting but, in my personal opinion, they are a solution to a problem that no one really has. If they work for you, that's great 🙌 But in the communities I participate in that have tried them, they haven't been widely used.

ashfurrow avatar Dec 29 '20 16:12 ashfurrow

hfghfgh

cesarvarela avatar Aug 22 '21 08:08 cesarvarela

Yes Cesar?

pvinis avatar Aug 22 '21 08:08 pvinis

LOL just testing, I'm looking how to add comments to my blog!

cesarvarela avatar Aug 22 '21 08:08 cesarvarela