mikeizbicki / cmc-csci040

Computing for the Web
37 stars 58 forks source link

Project 04 Q #285

Closed JackPotter7 closed 1 year ago

JackPotter7 commented 1 year ago

Hi Mike, in regard to FIXME (task 0): get a list of all of the comments in the submission should we return a list with all of top-level comments, or all of the comments (including replies to other comments)?

Thanks and hope you feel better!

mikeizbicki commented 1 year ago

This should be all comments, including replies, and not just top level comments.

JackPotter7 commented 1 year ago

Hi @mikeizbicki another Q: regarding FIXME (task 3): filter the not_my_comments list to also remove comments that you've already replied to. Does this mean we should only filter the list to comment without ANY replies, or just comments that don't have any replies from our bot...

mikeizbicki commented 1 year ago

Keep all comments that don't have any replies from your bot. The lists you are creating in these first tasks are going to contain the comments that you will eventually reply to, and you need to be able to reply to a comment if someone else has replied to it, just not if you have replied to it.

JackPotter7 commented 1 year ago

@mikeizbicki great. One more further clarification (maybe this comes from my inexperience with reddit). Taking a look at the code below (my bot is masterfulcoolbot), would this mean I would also need to reply to the "hello hello" comment, or does the reply to the "hello hello hello" comment count?

Screen Shot 2022-11-16 at 7 52 51 PM
mikeizbicki commented 1 year ago

You don't need to reply to the "hello hello" comment, but you need to be able to reply to that comment.