Hi Evan, On Sun, 2010-03-14 at 12:30 -0500, Evan Prodromou wrote: > Dave Hall wrote: > > I thought I would bump this thread and see if anyone has any comments. > > I am reluctant to put in a lot of effort if there is no support for it > > from the core developers. > > > I would be too! I'm happy with the idea on this; let's see about the > details.
Thanks for the feedback on this one. Sorry for the lag, been focused on other projects for the last week or so. > >> replyTo is determined by just checking the plain version of the message > >> with preg_match('/^@([A-Za-z0-9]{1,64})/') and the user id will be > >> passed into whoGets by addToInboxes() > >> > Don't do that. Use getReplies(), which is where all the smarts are. The reason I propose doing it this way is that there is a convention on twitter that if you want a reply to go to all of your followers you prefix it with ".", so "@evan thanks for the feedback" would only go to you (and any one else who follows both of us), while "....@evan thanks for the feedback" would go to all of my followers. Doing it this way preserves this functionality. > Otherwise, this seems fine. The big thing to think about is that changes > to the core DB schema aren't allowed except on minor release numbers. So > if you implement this in core, it'd have to wait until 1.0. That's ok, I'd would have liked to have got it into a dot release, but understand. > If you do it as a plugin, you might want to add a separate table for the > reply prefs. I was thinking about doing it in core, would you prefer it to be a plugin? I think it is functionality a lot of people would (may?) like to see in statusnet and identi.ca Cheers Dave _______________________________________________ StatusNet-dev mailing list StatusNet-dev@lists.status.net http://lists.status.net/mailman/listinfo/statusnet-dev