Closed Bug 976653 Opened 10 years ago Closed 9 years ago

Missing opengraph in the app page

Categories

(Marketplace Graveyard :: Consumer Pages, enhancement, P2)

Avenir
enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mte90net, Assigned: mstriemer)

References

Details

(Whiteboard: [repoman][ktlo])

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:29.0) Gecko/20100101 Firefox/29.0 Iceweasel/29.0a2 (Beta/Release)
Build ID: 20140214004003

Steps to reproduce:

Share the link on the social network (ex: Facebook or Google+)

Example link: https://marketplace.firefox.com/app/ringtone-picker



Actual results:

The image loaded it's the marketplace icon, the title link is Firefox Marketplace and missing a short description of the app.



Expected results:

The image loaded as the app logo, the title link as the name of the app and a short description.


This feature it's very important for share the apps, now it's impossible share correctly the app for a better advertising.
Status: UNCONFIRMED → NEW
Component: General → Consumer Pages
Ever confirmed: true
Check if the page is a detail page in Django and add the OG tags, problems with caching? Marking as P3.
Priority: -- → P3
Severity: normal → enhancement
Blocks: 1016433
Assignee: nobody → kngo
Whiteboard: [repoman]
https://github.com/mozilla/zamboni/pull/2596
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Note, we can do this for feed detail pages too.
when this integration will be present on the official website?
We ship every Tuesdays around noon PST, so, soon.
I'm not seeing the OG tags so there's a bug somewhere.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Hi all,
I wanted to weigh in, from a marketing perspective, we are now running between 2-3 marketplace marketing campaigns every month, each campaign has a social component (on Facebook). The idea is to expose as many users as possible to apps and app collections and have users share apps with their friends, being able to do so, pointing to a specific app, would be helpful and supportive of marketing efforts.

Thanks!
Kevin: I just tested locally, your code works fine, the problem is at the nginx level.

https://github.com/mozilla-services/svcops-puppet/blob/dev/modules/marketplace/templates/nginx/marketplace.conf#L282

The rule for app detail page is:
 rewrite "^/app/(?![0-9a-f]{8}(-[0-9a-f]{4}){3}-[0-9a-f]{12}/manifest\.webapp$).*" /server.html break;

(Confusing rule - ignore everything inside the parenthesis, because of the '?' - for the detail page, the rule simplifies to "^/app/.*")

That rule rewrites the app detail page url to simply /server.html, so in zamboni, it does not resolve to the detail page url and therefore we never add the opengraph stuff. We should get ops to do a separate rule in nginx just for the detail page that keeps the uri intact ; I don't know if this has other consequences though.
Flags: needinfo?(kngo)
Severity: enhancement → normal
Priority: P3 → P2
Blocks: 1135257
No longer blocks: 1135257
Blocks: 1135257
Severity: normal → enhancement
Whiteboard: [repoman] → [repoman][ktlo]
https://github.com/mozilla-services/svcops-puppet/pull/119
Status: REOPENED → ASSIGNED
Flags: needinfo?(kngo)
Assignee: kngo → mstriemer
Apps now have opengraph data on dev.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.