flpvn / wittytwitter

Automatically exported from code.google.com/p/wittytwitter
0 stars 0 forks source link

Can't mark tweet as favorite #214

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. right click a tweet, no favorite option
2.
3.

What is the expected output? a "Market Tweet Favorite" option

What do you see instead? zilch, nada, nuthin

Original issue reported on code.google.com by keith.el...@gmail.com on 3 Feb 2009 at 6:32

GoogleCodeExporter commented 9 years ago
heres a patch for TwitterLib with methods to Create and Destroy Favorites.  I've
tested these and they work however they fail if status updates is already
afavorite(when creating) or is not already a favorite(when destroying).  I've 
started
on the ui updates but need to workout dome details, figured id upload this now.

Original comment by sean.ste...@gmail.com on 10 Feb 2009 at 5:59

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks Sean for putting this out.  

Original comment by keith.el...@gmail.com on 10 Feb 2009 at 1:53

GoogleCodeExporter commented 9 years ago
Here's the rest of it.  "Add Favorite" and "Remove Favorite" are available from 
the
context menu for recent,replies, and user.  checks to if favorite is already 
there
before adding/removing.  I don't really like the way I've done the paging in
GetFavorites(int userId).  I originally had it using 
GetUser(userId).FavoriteCount
however that value seemed to be unreliable(i think its a problem with the api 
caching
the value, but I wasn't 100% sure).  So I went with the method of checking the 
return
count and if its 20, checking the next page.

Original comment by sean.ste...@gmail.com on 10 Feb 2009 at 4:57

Attachments:

GoogleCodeExporter commented 9 years ago
Issue 5 has been merged into this issue.

Original comment by james.deville on 31 Mar 2009 at 6:16