Last Comment Bug 787272 - disable recommend button on logout
: disable recommend button on logout
Product: Firefox
Classification: Client Software
Component: SocialAPI (show other bugs)
: unspecified
: x86 Mac OS X
: -- normal (vote)
: Firefox 18
Assigned To: Shane Caraveo (:mixedpuppy)
: Shane Caraveo (:mixedpuppy)
Depends on: 789248
  Show dependency treegraph
Reported: 2012-08-30 17:08 PDT by Shane Caraveo (:mixedpuppy)
Modified: 2012-09-25 11:39 PDT (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

hide share when logged out (2.45 KB, patch)
2012-08-31 11:12 PDT, Shane Caraveo (:mixedpuppy)
jaws: review+
Details | Diff | Splinter Review
minor fix (2.48 KB, patch)
2012-09-06 13:48 PDT, Shane Caraveo (:mixedpuppy)
mixedpuppy: review+ approval‑mozilla‑aurora+
Details | Diff | Splinter Review

Description Shane Caraveo (:mixedpuppy) 2012-08-30 17:08:51 PDT
providers can specify logged in/out state through the social.user-profile message.  We should only enable the recommend button when we have a logged in state.  Currently, if you "logout" the recommend button is still available.
Comment 1 Shane Caraveo (:mixedpuppy) 2012-08-31 11:12:59 PDT
Created attachment 657368 [details] [diff] [review]
hide share when logged out

[Approval Request Comment]
social share button should not be displayed if the user is not logged in
Comment 2 :Gavin Sharp [email:] 2012-08-31 11:26:32 PDT
Comment on attachment 657368 [details] [diff] [review]
hide share when logged out

Best to avoid asking for aurora approval until you have r+
Comment 3 Shane Caraveo (:mixedpuppy) 2012-09-04 12:45:15 PDT
Comment 4 Shane Caraveo (:mixedpuppy) 2012-09-04 12:45:48 PDT
Comment on attachment 657368 [details] [diff] [review]
hide share when logged out

[Approval Request Comment]
need to remove share button when user is not logged in
Comment 5 Lukas Blakk [:lsblakk] use ?needinfo 2012-09-05 15:53:44 PDT
We don't approve for aurora until patches have baked on central for a bit, set 'checkin-needed' in the keywords if you need someone to land this to trunk.
Comment 6 Shane Caraveo (:mixedpuppy) 2012-09-06 13:42:02 PDT
a new try to verify that tests are fixed by bug 789248, which is also showing up under bug 785676.
Comment 7 Shane Caraveo (:mixedpuppy) 2012-09-06 13:48:05 PDT
Created attachment 658984 [details] [diff] [review]
minor fix

minor fix on previous patch, carry forward r+
Comment 8 Shane Caraveo (:mixedpuppy) 2012-09-06 15:40:43 PDT
try works with patch in bug 789248
Comment 9 :Felipe Gomes (needinfo me!) 2012-09-06 16:32:26 PDT
Comment 10 :Gavin Sharp [email:] 2012-09-06 20:47:06 PDT
Comment on attachment 658984 [details] [diff] [review]
minor fix

[Triage Comment]
Comment 11 Ed Morley [:emorley] 2012-09-07 08:47:45 PDT
Comment 12 :Gavin Sharp [email:] 2012-09-25 11:39:22 PDT

Note You need to log in before you can comment on or make changes to this bug.