1116786 - Add trending sort option to search API

Status: VERIFIED FIXED

(Marketplace Graveyard :: API, defect, P3)

Description

[Rob Hudson [:robhudson]]

One ES has trending in it (bug 1116342) let's add this to the API.

Comment 1

[Rob Hudson [:robhudson]]

https://github.com/mozilla/zamboni/commit/1a99f8d 

To verify will require testing the API since this isn't yet exposed in the UI.

* Test that the search API allows to sort by 'trending'
* Test that the top 5 or 10 apps returned when sorting by trending are actually trending -- meaning, their install stats are up in the past week vs the prior weeks 2-4.

Comment 2

[Madalin Cotetiu]

Verified in FF37(Win7) in marketplace-dev.allizom.org

I have performed the following request:
GET https://marketplace-dev.allizom.org/api/v2/fireplace/search?sort=trending
After that I have analyzed the stats of the returned apps. 

apps:
LingQ: http://screencast.com/t/oLQnt03ahYou
Do you speak ... ? http://screencast.com/t/8hdd0JgZSA
Yohoho! http://screencast.com/t/jtwx1wEX
KanaFire: http://screencast.com/t/bmTBiuBRUmGj
BzDeck: http://screencast.com/t/WumIKs0c

So all the apps in the returned list have a growth in installs in the last week.
Anything else we should check regarding this issue?

Comment 3

[Rob Hudson [:robhudson]]

(In reply to Madalin Cotetiu from comment #2)
> So all the apps in the returned list have a growth in installs in the last
> week.
> Anything else we should check regarding this issue?

This is perfect. Thanks!

Comment 4

[Madalin Cotetiu]

Closing this bug.