Status

Marketplace
API
P3
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cvan, Assigned: robhudson)

Tracking

2013-05-23
Points:
---
Dependency tree / graph

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
In payments land, we allow you to advertise from the detail page of a free app a link to a paid version of your app.

On the detail page of my free app (https://marketplace-dev.allizom.org/app/carrier-info/), I advertise a paid version of my app:

http://f.cl.ly/items/2a3b0w0U1U0F3E0K010n/Screen%20Shot%202013-05-01%20at%2010.46.43%20PM.png

Michael, how should this look in the new designs?

Comment 1

5 years ago
To whomever implements this feature: make sure you do models('app').cast(upsell). It might have to be done at the view layer. If we don't do it, clicking on the upsell will cause an uncomfortable amount of waiting for no reason.
Created attachment 746018 [details]
Upsale

Mocks:

https://www.dropbox.com/s/lwhjlwv76t2wp07/Marketplace_Explore_June_v11.zip
Created attachment 746019 [details]
Desktop Upsale
Priority: -- → P3
(Reporter)

Updated

5 years ago
Summary: Upsell missing in Fireplace/new designs → Add upsell to detail page
(Assignee)

Comment 4

5 years ago
Since we're moving towards keeping data in elasticsearch to avoid hitting the database on consumer pages, let's consider what minimal fields we'll need for what the API returns and not a whole app resource object.

For example, if we could simply store ID, icon, and name (and resource URI for the app detail API could be generated from the ID), that'd be perfect. Then we can keep that in ES along with the app and that's our upsell data.
(Assignee)

Updated

5 years ago
Blocks: 875012
(Assignee)

Comment 5

5 years ago
API side is done. Fireplace may need a small update if we changed anything they're using.

https://github.com/mozilla/zamboni/commit/b3ec3e3
Assignee: nobody → robhudson.mozbugs
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2013-05-23

Comment 6

5 years ago
Krupa: do you have any apps on -dev with upsells that we can test with?

Comment 7

5 years ago
Pushed unstyled upsells:

https://github.com/mozilla/fireplace/commit/d7e6194fcaa0426da296a0df31ca5752388f0b30

You can test with the app slug "upsell" when using flue
(Reporter)

Comment 8

5 years ago
As Basta mentions in comment 7, these are unstyled. Reopening for styles...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 9

5 years ago
Test app: https://marketplace-dev.allizom.org/app/test-app-cony8368/
(Assignee)

Comment 10

5 years ago
Making this the API side of the bug.
Created bug 877892 for fireplace.
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Component: Consumer Pages → API
Keywords: uiwanted
Resolution: --- → FIXED
Summary: Add upsell to detail page → Add upsell to API
Whiteboard: [fireplace]
You need to log in before you can comment on or make changes to this bug.