1351577 - Integrate LeanplumPushReceiver/LeanplumPushListenerService

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, enhancement, P1)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

To receive and process push messages we need to integrate LeanplumPushReceiver into the app. Normally that's just adding it to the manifest. But in our case we are using GCM push for implementing web push already. Instead we will need to route incoming push messages to LeanplumPushListenerService or our GcmMessageListenerService.

Additionally we need to pass the sender ID to LeanplumPushService.

Comment 1

[Nevin Chen(Not active on Bugzilla)]

yes... That's the thing I'm goiing to work on later. Since Push Notificaiton is not in the scope (55) yet... right?

Comment 2

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

Correct.

Comment 3

[Joe Cheng [:jcheng] (please needinfo)]

right not in scope for 55

Comment 4

[Nevin Chen(Not active on Bugzilla)]

This is not in scope for 56 either. Maybe we can create another meta bug and link this bug to there.

Comment 5

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

[triage] Bulk edit: product thinks leanplum is high priority: P1 rank 1.

Comment 6

[Barbara Bermes [:barbara] - NI please!]

Mike, can you please confirm what the status of this feature is. Has this ever landed?

Also looking at this: https://bugzilla.mozilla.org/show_bug.cgi?id=1397208

Comment 7

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

> Mike, can you please confirm what the status of this feature is. Has this ever landed?

I think so: there is a LeanplumPushReceiver [1] and LeanplumPushService [2] in the code but I can't be certain with digging in more to fully understand what we implement with Leanplum, which is outside the scope of my role as triage owner (i.e. whoever implements Leanplum would be a better person to do this). fwiw, these classes were added in bug 1351585 (add SDK), bug 1386151 (enable push feature), and bug 1392224 (make fennec default handler for leanplum push).

Action item for this bug: did we implement everything set out in comment 0 in these other bugs? If so, please dupe.

[1]: https://searchfox.org/mozilla-central/source/mobile/android/thirdparty/com/leanplum/LeanplumPushReceiver.java
[2]: https://searchfox.org/mozilla-central/source/mobile/android/thirdparty/com/leanplum/LeanplumPushService.java

Comment 8

[Sebastian Kaspari (:sebastian; :pocmo)]

Yes, if we use push with leanplum then this has already been implemented (at least that's what I filed this bug for back then).