Closed
Bug 645095
Opened 14 years ago
Closed 14 years ago
Facebook badges: Add mobile badge
Categories
(Websites :: Facebook, defect)
Websites
Facebook
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: williamr, Assigned: allia.z)
References
()
Details
Attachments
(2 files)
|
15.62 KB,
image/png
|
Details | |
|
3.15 KB,
patch
|
Details | Diff | Splinter Review |
Please replace the 'make the web better' badge with the attached mobile badge.
Zach, can we also track the number of mobile badge shares separately? That's a pretty important stat for us to have.
| Reporter | ||
Comment 1•14 years ago
|
||
| Assignee | ||
Comment 2•14 years ago
|
||
Fix for this and 645082 (Adds mobile badge and mobile download link)
| Reporter | ||
Comment 3•14 years ago
|
||
Thanks Zach! Is there any way we can track the number of mobile badge shares as mentioned in comment 0? That's very important for us.
| Assignee | ||
Comment 4•14 years ago
|
||
We can, do you have a stats url I can ping every time one is posted? I can send the name of the badge that was posted.. If not, what stats would you prefer me using?
| Reporter | ||
Comment 5•14 years ago
|
||
We have the WebTrends tags b1 - b5 set up for the badges. Laura, can you confirm this approach is okay or suggest another option? Let's chat in the morning. Thanks!
Here's the URL format:
http://www.mozilla.com/?WT.mc_id=b1&WT.mc_ev=click
Comment 6•14 years ago
|
||
Just checking in ... when will this be ready for QA to start poking at?
| Reporter | ||
Comment 7•14 years ago
|
||
Fred, can you please apply Zach's patch from comment 2 so we can verify on stage? Thanks!
Zach, Laura said we could try pinging mozilla.com in the backround with the tags badge_1 - badge_4 for the 4 different badge designs. Note that we don't want the user to actually be redirected or shown this page when posting a badge.
The syntax would be:
http://www.mozilla.com/?WT.mc_id=badge_1&WT.mc_ev=click
Updated•14 years ago
|
Attachment #522287 -
Attachment is patch: true
Attachment #522287 -
Attachment mime type: application/octet-stream → text/plain
Comment 8•14 years ago
|
||
(In reply to comment #7)
> Fred, can you please apply Zach's patch from comment 2 so we can verify on
> stage? Thanks!
Oh absolutely.
trunk: r86649.
stage: r86650.
I removed the existing mobile.png badge image and replaced it with the one attached to this bug (please correct me if that was a mistake).
Comment 9•14 years ago
|
||
The test app on Facebook does not seem to point to stage anymore, is that intentional?
| Reporter | ||
Comment 10•14 years ago
|
||
(In reply to comment #9)
> The test app on Facebook does not seem to point to stage anymore, is that
> intentional?
The 'test' app was converted to the production app, so we need a new staging app for testing. I've set one up that we can use for testing.
Fred, I believe you just need to add the staged app's secret key on mozilla.com stage. I'll send it to you privately and then we can test the mobile update changes on stage.
Note: When the Facebook badges update is pushed to mozilla.com tomorrow morning, we will still be using the current production app. That is to say, the new stage app will *not* convert to the production app like we did for the desktop launch.
| Reporter | ||
Comment 11•14 years ago
|
||
Per comment 10, here's the new stage app + page:
http://www.facebook.com/pages/Firefox-4-mobile/181512511895753?sk=wall
Once bug 645893 is resolved, you can test the staged app here:
http://www.facebook.com/pages/Firefox-4-mobile/181512511895753?sk=app_135742499831827
Comment 12•14 years ago
|
||
Moving this to fixed and ready for QA per comment 11.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•