894643 - API consumers should be able to add collections to a region/page pair

Status: RESOLVED FIXED

(Marketplace Graveyard :: API, defect, P2)

Description

[Chuck Harmston [:chuck]]

From the publishing tool, publishers will be able to add an arbitrary number of collections and featured applications to a region (country + operator)/page (category + homepage) pair.

To do this, we need an endpoint that allows users to add collections to a region/page pair.

This endpoint should be protected by the Apps:Publisher permission created in bug 894425.

Comment 1

[Christopher Van Wiemeersch [:cvan]]

Flue API endpoint @ /api/v1/rocketfuel/collections/
https://github.com/mozilla/flue/commit/8498d80

Comment 2

[Christopher Van Wiemeersch [:cvan]]

Should we be adding collections to a region/carrier/collection_type pair? Or should we be having a separate collection for each region/carrier/collection_type? I'm thinking the latter.

It'd make bug 894591 the same as 894638, etc. There are several dupes we could dupe if we go this route.

Comment 3

[Mathieu Pillard [:mat]]

It looks like all that will be left to do once bug 903489 is done is to determine, code and test the relevant unique constraints.

Comment 4

[Mathieu Pillard [:mat]]

So, unique constraints are going to be tricky, because we'd want constraints that depend on the value for another field. So we are going to have to do this manually:

<cvan> (...) let's just make sure the admin doesn't create more than one category/region/carrier combination for operator shelves+featured apps

Worst case scenario, an extra collection would be created, we need to make sure that a) the consumer pages only display the first one b) the admin displays all of them to be able to edit and remove the extra one.

Comment 5

[Mathieu Pillard [:mat]]

Fixed in https://github.com/mozilla/zamboni/commit/f280e892d1ea962641c145ca1ecb2b15236251c7