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

Reply via email to