If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Apps showing as unavailable

VERIFIED FIXED

Status

Marketplace
API
P1
major
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: andym, Assigned: robhudson)

Tracking

x86
Mac OS X
Points:
---

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
1. Open up stage marketplace on a b2g phone
2. Pick a region with some apps in it, eg: Spain
3. Search for :paid

Expected:

Can purchase an app

Actual:

Everything shows as unavailable. Clicking on the app on going to details shows the same.

Making a P2 because this blocks testing on stage of paid apps.
Something wrong with the search API / ElasticSearch it looks like:

% curl https://marketplace.allizom.org/api/v1/apps/search/\?q\=pricetier6-1
[...]
     "price_locale" : null,
     "price" : null,
[...]

% curl https://marketplace.allizom.org/api/v1/apps/app/pricetier6-1/
[...]
   "price_locale" : "$2.99",
   "price" : "2.99",
[...

Comment 2

4 years ago
We lost a day of testing in Hungary over this. Making this a P1
Priority: P2 → P1
Rob and Mat were both working on this.  Whoever gets to it first can keep it. :)  I think Mat is asleep so -> rob.
Assignee: nobody → robhudson.mozbugs
(Assignee)

Comment 4

4 years ago
This commit changed it:
https://github.com/mozilla/zamboni/commit/62354cc609e477f78a591f4f3f34fa191c800505

Prior to this we passed the `region`, the integer, into es_app_to_dict which was used for the price lookups.

After this commit the passed in region is no longer used and the region is pulled from the GET parameter as the region slug.

Passing the region slug to the price lookups is essentially what broke this, as they now return `None`.

Since no other code passes in the region ID except the search calls I'm going to remove this parameter from this method. The region will be set from the GET param or fall back to the region set in the middleware and pulled via `get_region()`. The GET param override is useful from the collections tool.

The inconsistency between some calls wanting a region slug and others wanting a region ID is unfortunate.

PR coming soon.
(Assignee)

Comment 5

4 years ago
PR: https://github.com/mozilla/zamboni/pull/1612
(Assignee)

Comment 6

4 years ago
Merged: https://github.com/mozilla/zamboni/commit/57ef65f 

This will need to be pushed to stage for testing.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Comment 7

4 years ago
Verified as fixed in https://marketplace.allizom.org/ on FF29 (Android).
Attaching postfix screenshot.
Closing bug.
Status: RESOLVED → VERIFIED

Comment 8

4 years ago
Created attachment 8358383 [details]
postfix paid apps android.png
You need to log in before you can comment on or make changes to this bug.