Closed Bug 1353793 Opened 8 years ago Closed 8 years ago

Only call LeanPlum.start for Certain Language/Locales

Categories

(Firefox for iOS :: Telemetry, enhancement, P1)

Other
iOS
enhancement

Tracking

()

RESOLVED FIXED
Iteration:
1.20

People

(Reporter: sdaswani, Assigned: sdaswani)

References

Details

(Whiteboard: [MMA] [MobileCore])

Attachments

(1 file)

55 bytes, text/x-github-pull-request
farhan
: review+
Details | Review
Since we have a monthly MAU limit, we can't call LeanPlum.start for all Firefox installs, but have to limit it by language or locale. This is assigned to Susheel until he figures out with Product what the language/limits should be.
Assignee: nobody → sdaswani
Blocks: 1351446
Whiteboard: [mma]
The wrapper should also satisfy this requirement when instantiating LP: 'Use a random identifier for the Device ID, rather than IDFA.'
Only enable LP.start for the following cases: Locale is US, language is EN-US Locale is DE, language is DE Locale is UK, language is EN-UK Locale is CA, language is EN-US Locale is AU, language is EN-AU Locale is HK, language is ZH-TW Locale is SG, language is EN-US Barbara please confirm these make sense.
Flags: needinfo?(bbermes)
To be clear, we have 4 options here, and I need info and ultimately a decision from Barbara: 1) Wait for location access from users through WebApp usage. Barbara needs to let me know what % of FF iOS users grant the App location access. 2) Use a combination of Locale/Language to determine the region. This is easy to do but will result in false positives. 3) Parse the App Store receipt to determine what App Store the app was downloaded from. This is not super easy to do and will result in false positives, but probably less than option 2. 4) Use Mozilla Location services. This is mediufm effort to do but will get us accurate location, per 1). Note that whatever option is chosen, specific country campaigns will be available in all cases.
Summary: Create Wrapper for LeanPlum that only calls LeanPlum.start for Certain Language/Locales → Only call LeanPlum.start for Certain Language/Locales
Priority: -- → P2
Whiteboard: [mma] → [MMA] [MobileCore]
Iteration: --- → 1.20
Priority: P2 → P1
Attached file Pull Request
Attachment #8862210 - Flags: review?(fpatel)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Attachment #8862210 - Flags: review?(fpatel) → review+
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
Flags: needinfo?(bbermes)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: