868254 - Don't list apps under price tier 10 for countries that don't support carrier billing

Status: RESOLVED FIXED

(Marketplace Graveyard :: API, defect, P2)

Description

[krupa raj[:krupa]]

The minimum price tier supported for credit card billing is price tier 5(David Bialer to confirm). Price tiers below 5 can only have carrier billing


If the country doesn't support carrier billing, we should not be listing apps which have price tiers (0-4)

This is bad since it will discourage developers from selling their apps at cheaper, affordable rates.

(Also, this sucks)

Comment 1

[David Bialer [:dbialer]]

It should be price tier 4 - $.99.  Note, that this only affects a single country right now that is not doing carrier billing, only credit cards.  The minimum price we can charge for a credit card is $.99, .99 Euro or GBP.

Comment 2

[Wil Clouser [:clouserw]]

From an engineering POV, I think a "supported price tiers" property on each region constant would make this flexible enough for the future.

Comment 3

[David Bialer [:dbialer]]

Bingo

Comment 4

[krupa raj[:krupa]]

Oh, I forgot. With the new price tiers, we don't have a price tier 4. The new price tiers look something like http://cl.ly/image/2u2a0g1x1i1d

So, we should we not showing credit cards for apps with less than price tier 10

Comment 5

[Matt Basta [:basta]]

This is an API thing since we get our listings from the API now.

Comment 6

[Allen Short [:ashort]]

The "price_tier" field in elasticsearch for apps is currently a string, so doing range comparisons on it doesn't seem to work currently. Someone more knowledgeable about ES than me should comment on what it'd take to change that.

Comment 7

[Rob Hudson [:robhudson]]

Price tier comes from obj.addonpremium.price.name. If the goal is to not show apps that only have a price tier that equates to carrier billing we can add something to the index that computes this before indexing. E.g. carrier_billing_only=True or something? If price tiers change again this can be updated pretty easily.

Comment 8

[Andy McKay]

We should probably add this information to the price tier API.

Comment 9

[Allen Short [:ashort]]

https://github.com/mozilla/zamboni/commit/cfd8a80

Comment 10

[krupa raj[:krupa]]

https://bugzilla.mozilla.org/show_bug.cgi?id=877555 blocks me from verifying this bug.