search

kylewm / redwind

Flask-based personal website #IndieWeb #OwnYourData
Other
45 stars 5 forks source link

Reply to all participants in a Twitter conversation #70

Closed LanaCoyote closed 9 years ago

LanaCoyote commented 9 years ago

This PR automatically replies to all mentioned users in a status rather than just the author of the status. This allows a user to reply to Twitter conversations without having to manually include mentioned users.

kylewm commented 9 years ago

Ah, good idea. What do you think about automatically adding in @-names if and only if no @-names are mentioned in the status? That way if you just want to reply to one or two people in a thread, you can list them explicitly.

LanaCoyote commented 9 years ago

Maybe only if there's no @-names at the start of a post, rather than in the entire body? Otherwise it sounds good.

kylewm commented 9 years ago

hmm, to avoid false positives where someone unrelated to the conversation is mentioned in the reply? Maybe in that case, we should say, auto-prepend the @-names unless one or more of them is already mentioned in the status.

LanaCoyote commented 9 years ago

That makes the most sense. I'll work on it later tonight

LanaCoyote commented 9 years ago

Ok, I made it so it checks to see if any of the participating users are already mentioned in the status.

kylewm commented 9 years ago

Looking forward to trying this out, Thanks! (and your way of excluding the current user is much better than the quick-and-dirty BLACKLIST thing I was doing, so thank you for fixing that too)

I don't now what was going on with the merge conflicts, but I fixed them on the command line.

LanaCoyote commented 9 years ago

Most likely related to my repo being out of date. I'll have to fast forward it before I do anything else