Closed
Bug 1397188
Opened 7 years ago
Closed 7 years ago
[LeanPlum] - Push Notifications also received for active users
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox57 verified)
VERIFIED
FIXED
Tracking | Status | |
---|---|---|
firefox57 | --- | verified |
People
(Reporter: bsurd, Assigned: cnevinchen)
References
Details
(Whiteboard: [FNC][SPT57.3][INT][LP_M2])
Device: - Samsung Galaxy Note 4 (Android 5.0.1); - Nexus 5 (Android 6.0.1); - Nexus 6 (Android 7.1.1); - Honor 8 (Android 7.0). Build: - Nightly 57.0a1 (05-Sep-2017); Prerequisites: - install Scitchboard:Experiments and enable LeanPlum Steps to reproduce: 1. Open Fennec and setup the device to receive any of the Campaign Push Notifications; 2. Do not close the app and wait. Expected result: No push notification is received with the app open. Actual result: A push notification is received even if the app is in the foreground.
Assignee | ||
Comment 1•7 years ago
|
||
In the test, we send push after 5 minutes after "E_Launch_Browser" event. We don't know if the user is still running the app in the foreground or keep our app opened in the background. In the real world, we'll send this push for users that start our app many(e.g.14) days ago. In this scenario, normally the user won't have our app in the background.But if we really want to avoid this, maybe we can use " Last active time is prior? 5 minutes ago" in message targets?
Assignee | ||
Updated•7 years ago
|
tracking-fennec: ? → +
Comment 2•7 years ago
|
||
Bogdan, given the latest offline clarification that it's in fact an Android limitation (push notification for active users), maybe you would like to verify the latest behavior with the "last active" Nevin has configured in the Leanplum dashboard, before the sing-off? Thanks.
Flags: needinfo?(bogdan.surd)
Reporter | ||
Comment 3•7 years ago
|
||
From the testing we have conducted today this fix seems to work quite well. Marking this as verified.
Status: NEW → RESOLVED
tracking-fennec: + → ---
Closed: 7 years ago
Flags: needinfo?(bogdan.surd)
Resolution: --- → FIXED
Updated•7 years ago
|
Whiteboard: [FNC][SPT57.3][INT]
Updated•7 years ago
|
Whiteboard: [FNC][SPT57.3][INT] → [FNC][SPT57.3][INT][LP_M2]
Reporter | ||
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•