812224 - Enable the purchase button on B2G desktop

Status: RESOLVED FIXED

(Marketplace Graveyard :: Payments/Refunds, defect, P3)

Description

[Kumar McMillan [:kumar]]

Paid apps are only supported on B2G (at first) but currently this is not detected correctly for B2G desktop. We need to support B2G desktop because it's an easier way to develop.

STR:
- Enable payments on a dev server using the waffle switch (set disable-payments to False)
- Edit an app to make it premium
- Visit the app details page on B2G desktop

Result: the purchase button is grayed out with a message saying "you must be on Firefox OS"

The current workaround for this is to enable middleware that forces mobile for all requests but we can't use this in production obviously. https://github.com/mozilla/zamboni/blob/84364e00cf19e2fd060a99e04d26c65a99769f44/sites/altdev/settings_mkt.py#L11

Comment 1

[Kumar McMillan [:kumar]]

cvan or potch: one of you probably knows how to fix this. I took a quick look but couldn't figure out a way.

Comment 2

[Wil Clouser [:clouserw]]

Make sure desktop with a regular browser remains disabled please/thanks.

Comment 3

[Potch [:potch]]

The best solution here is to use r2d2b2g instead of stock B2G Desktop. r2d2b2g changes the user agent to match firefox OS (No clue why b2g desktop doesn't).

Comment 4

[Kumar McMillan [:kumar]]

Hmm, I can't see anything that the simulator is doing differently to define a user agent https://github.com/mykmelez/r2d2b2g/blob/master/addon/lib/main.js#L703 It just starts a subprocess with b2g desktop. We will probably need to support custom builds of desktop b2g before features land so maybe myk or Harold can show us how they got the UA string in there.

Comment 5

[Kumar McMillan [:kumar]]

we need to come up with a solution for this so we can disable the middleware that forces mobile https://github.com/mozilla/zamboni/commit/d67edf90bf567537816b56041d9491e2d410eac9

potch, I think you're back next week? If not, I can grab it.

Comment 6

[Kumar McMillan [:kumar]]

Yay. I have a fix for this. It requires some custom prefs to override the desktop UA which I will document. I'll also remove the middleware from altdev when it's working.

Comment 7

[Kumar McMillan [:kumar]]

This was fixed in bug 821000 (UA string on desktop)