599995 - The sent tweets lack the @Recipient part

Status: VERIFIED FIXED

(support.mozilla.org :: Army of Awesome, task)

Description

[David Tenser [:djst]]

When responding to a tweet and picking a snippet, the "@RecipientUserName " prefix isn't added to the "Get personal" text box or the resulting tweet.

Comment 1

[Alex Buchanan [:abuchanan]]

(In reply to comment #0)
> When responding to a tweet and picking a snippet, the "@RecipientUserName "
> prefix isn't added to the "Get personal" text box or the resulting tweet.

It uses twitter's 'reply_to_status_id' so it's posted directly to their wall.  I can still add this if you think it's necessary though (remember it cuts down on allowed text).  Let me know.

Comment 2

[David Tenser [:djst]]

I have no problem with dropping the @RecipientUserName part if it isn't needed anymore, but the big issue I have is that it's not possible to tell who you were replying to either. 

Normally under a tweet, you see e.g. "less than 10 seconds ago via custcare *in reply to RecipientUserName*"

But the tweets sent by this app doesn't include the "in reply to" part either, making your tweet seem like a normal, non-reply tweet to your own wall instead, which is kind of weird.

Is the "in reply to" part connected to the @RecipientUserName text of the tweet itself?

Comment 3

[David Tenser [:djst]]

To clarify my previous comment: if there's no way to tell to whom the message was sent, it just looks like you're talking to yourself in your own Twitter feed, which I don't like.

So, either add the @Recipient part or add the "in reply to Recipient" in the meta/status line under the tweet. If they're connected, please add both.

Comment 4

[Alex Buchanan [:abuchanan]]

Ok, cool, thanks.  Tweets are being posted to your own wall, which doesn't seem right.  

In the API I'm using update_status(),
http://joshthecoder.github.com/tweepy/docs/api.html#status-methods

But I think what I want instead is a direct message,
http://joshthecoder.github.com/tweepy/docs/api.html#direct-message-methods


I'll try that out

Comment 5

[David Tenser [:djst]]

Aren't direct messages private? If so, we do not want that. We want this to be public.

Comment 6

[Alex Buchanan [:abuchanan]]

adding @recipient does the trick, thanks for clarifying that.

I need to rebase a bunch of commits and push them all the master, I'll post here when I do.

BTW, I'm removing the hard-coded debug line, so from now on, **all twiiter replies from stage will go to actual users! ** so be careful with your testing.

Comment 7

[Alex Buchanan [:abuchanan]]

http://github.com/jsocol/kitsune/commit/c7542fbba473319217f29b3d4118b882dba5a037

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on http://master.support.mozilla.com/en-US/army-of-awesome.

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Post-fix screenshot of Twitter reply, posted from staging

Comment 10

[David Tenser [:djst]]

In the post-fix screenshot, I don't see the #fxhelp tag. The real fix for this is bug 600732 though, so I won't reopen this bug.