544906 - Add updateURL to personas json attribute

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect, P4)

Description

[Ryan Doherty (:rdoherty)]

Personas in Firefox 3.6 support an updateURL to ping for updates (and used for stats). It should be added so we can track daily users and update personas that are out of date.

Example:
{
"id":"81070",
"updateURL":"https://www.getpersonas.com/en-US/update_check/81070"
}

The full json format can be found by viewing a persona detail page and searching for 'persona='.

Comment 1

[Barry Chen]

Perhaps I'm misunderstanding the bug, but the persona attribute in the source of a persona's detail page on getpersonas.com already includes the "updateURL".  Searching for 'persona=' in the source of a persona's detail page on AMO doesn't give me any results.  I do see a data-browsertheme attr in the AMO page that has json for its value.  Is that where I'm adding the updateURL?

Comment 2

[Barry Chen]

Committed @ https://github.com/chenba/zamboni/commit/0786fd5ef897f2414c17b81363265cbb75966dce

Comment 3

[Barry Chen]

It's in jbalogh/master now: https://github.com/jbalogh/zamboni/commit/a67b305e800a5c2c07d328642177e0b570f97f2a

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED on next:

Next (contains "updateURL":"https://www.getpersonas.com/update_check/24265""): data-browsertheme="{"iconURL":"https://www.getpersonas.com/static/6/5/24265/preview_small.jpg","description":"","accentcolor":"#","header":"http://www.getpersonas.com/static/6/5/24265/Spltrfx.jpg","footerURL":"http://www.getpersonas.com/static/6/5/24265/SpltrfxF.jpg","textcolor":"#cccccc","id":"24265","category":null,"headerURL":"http://www.getpersonas.com/static/6/5/24265/Spltrfx.jpg","name":"SplatterFox Again","author":"gerryfx","footer":"http://www.getpersonas.com/static/6/5/24265/SpltrfxF.jpg","updateURL":"https://www.getpersonas.com/update_check/24265","previewURL":"https://www.getpersonas.com/static/6/5/24265/preview_large.jpg"}">

Prod (doesn't have it): data-browsertheme="{"iconURL":"https://www.getpersonas.com/static/1/6/16/preview_small.jpg","description":null,"accentcolor":"#f7fcff","header":"http://www.getpersonas.com/static/1/6/16/tbox-firefox_b.jpg","footerURL":"http://www.getpersonas.com/static/1/6/16/stbar-firefox_b.jpg","textcolor":"#","id":"16","category":"Firefox","headerURL":"http://www.getpersonas.com/static/1/6/16/tbox-firefox_b.jpg","name":"Firefox B","author":"Mozilla","footer":"http://www.getpersonas.com/static/1/6/16/stbar-firefox_b.jpg","previewURL":"https://www.getpersonas.com/static/1/6/16/preview_large.jpg"}">