Change lookup for a regions on a paid app for search

RESOLVED FIXED in 2013-09-24

Status

P2
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: andy+bugzilla, Assigned: andy+bugzilla)

Tracking

2013-09-24
x86
Mac OS X
Points:
---
Dependency tree / graph

Details

(Whiteboard: [qa-])

(Assignee)

Description

5 years ago
As part of the region rabbit hole discussion and bug 902185 we need to change how regions are looked up for an app. They should first:

- find all regions explicitly excluded for an app
- find the current price tier and find what regions are available for that app

Return a list of regions suitable for inclusions/exclusions. This could probably be part of the model or utils as we'll need to use it a bit. This would be used for search.
(Assignee)

Updated

5 years ago
No longer depends on: 902185
(Assignee)

Updated

5 years ago
Priority: -- → P2
Target Milestone: --- → 2013-09-10
(Assignee)

Comment 1

5 years ago
ttps://github.com/mozilla/zamboni/commit/9f8ca3
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

5 years ago
Was looking at the wrong bug.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Updated

5 years ago
Target Milestone: 2013-09-10 → 2013-09-17
We should make sure we're doing these exclusions on the detail page too.
(Assignee)

Comment 4

5 years ago
Bumped, will push after we tag on monday.
Target Milestone: 2013-09-17 → 2013-09-24
(Assignee)

Comment 5

5 years ago
https://github.com/mozilla/zamboni/commit/df6edd

Will need to do a full app index before QA'ing, but basically nothing should change, paid apps should all show up in the regions we'd expect. The difference is now, they should change as we change price tiers (the PriceCurrency object specifically).
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED

Comment 6

5 years ago
Please add STR here or mark it with [qa-] if no QA is needed.
Flags: needinfo?(amckay)
(Assignee)

Updated

5 years ago
Flags: needinfo?(amckay)
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.