612929 - Indicate which replies the user has replied to recently

Status: VERIFIED FIXED

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

Description

[William Reynolds [:williamr]]

Once a user has replied to a tweet, show this on the right side of the tweet in the tweet list by saying "You replied".

If others have also replied to that tweet, it should say "You and 1 other replied" or "You and X others replied". This mockup shows the styling and use cases:
http://people.mozilla.com/~williamr/communitycare/mockups/armyofawesome_231a.jpg

Since this is only useful to the user for a short time period, I think we could do this locally with a cookie and/or some JS. However, scrolling down to load more tweets, clicking the refresh button to reload tweets, or refreshing the page itself should still show the users which tweets they have replied to.

Comment 1

[Fred Wenzel [:wenzel]]

Here's some code that should do what you ask for:

https://github.com/fwenzel/kitsune/compare/master...cc-you-replied-612929

jsocol: r?

Notes:
- I only keep the last replied-to tweet number in localstorage. As it turns out, added replies show up immediately on page reload, so it didn't seem to make sense to me to keep a longer history there. Replies that come from the database and are shown on the page will be marked as mentioned above.
- When you make a reply and the window closes, your reply won't show up in the DOM tree yet. However, the tweet you replied to *will* be marked "You replied". Your actual reply shows up once you reload the page.

Comment 2

[James Socol [:jsocol, :james]]

master: https://github.com/jsocol/kitsune/commit/19ab09

Comment 3

[Fred Wenzel [:wenzel]]

Thanks, James!

Comment 4

[William Reynolds [:williamr]]

Appears the changes to show replies (this bug along with bug 612943 and bug 612935) are only on master and not support-stage. Can we get these added to support-stage? Thanks!

Comment 5

[Fred Wenzel [:wenzel]]

I cherry-picked all replies-related commits into 2.3.x:
https://github.com/jsocol/kitsune/compare/d5b28d1...3966d43

Comment 6

[Rebecca Billings [:rbillings]]

Verified on support-stage