Closed Bug 1452173 Opened 6 years ago Closed 6 years ago

Add Leanplum documentation for Users Opting out of pocket in top sites

Categories

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

All
Android
enhancement

Tracking

(firefox61 fixed)

RESOLVED FIXED
Firefox 61
Tracking Status
firefox61 --- fixed

People

(Reporter: mcomella, Assigned: petru)

References

Details

Attachments

(1 file, 1 obsolete file)

We have leanplum documentation in tree [1] that should be updated with the changes from bug 1445799.

Petru, can you take care of this?
Flags: needinfo?(petru.lingurar)
Yes please Petru, nice catch Mike.
Assignee: nobody → petru.lingurar
Flags: needinfo?(petru.lingurar)
Attachment #8966501 - Flags: review?(michael.l.comella)
Comment on attachment 8966501 [details]
Bug 1452173 - Add Leanplum documentation for Users Opting out of pocket in top sites;

https://reviewboard.mozilla.org/r/235224/#review241158

lgtm! Thanks for the quick turn-around! Documentation can be annoying but it's important so I really appreciate it! :)
Attachment #8966501 - Flags: review?(michael.l.comella) → review+
Pushed by michael.l.comella@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e041cf30962f
Add Leanplum documentation for Users Opting out of pocket in top sites; r=mcomella
Comment on attachment 8966501 [details]
Bug 1452173 - Add Leanplum documentation for Users Opting out of pocket in top sites;

https://reviewboard.mozilla.org/r/235224/#review241166

::: mobile/android/docs/mma.rst:67
(Diff revision 1)
>      "Focus Installed" -> true           // If Focus for Android is installed.
>      "Klar Installed" -> true            // If Klar for Android is installed.
>      "Pocket Installed" -> true          // If Pocket for Android is installed.
>      "Signed In Sync" -> true            // If the user has signed in to Mozilla account.
>      "Default Browser" -> true           // If the user has set Fennec as default browser.
> +    "Pocket in Top Sites" -> true       // If Top Sites home panel contain websites recommended by Pocket

Nit: this sounds kind of passive, maybe something like "If Pocket recommendations are enabled (by default or through user action)"
https://hg.mozilla.org/mozilla-central/rev/e041cf30962f
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 61
Attachment #8966915 - Attachment is obsolete: true
Attachment #8966915 - Flags: review?(sdaswani)
(In reply to Chenxia Liu [:liuche] - not actively working on Fennec from comment #6)
> Comment on attachment 8966501 [details]
> Bug 1452173 - Add Leanplum documentation for Users Opting out of pocket in
> top sites;
> 
> https://reviewboard.mozilla.org/r/235224/#review241166
> 
> ::: mobile/android/docs/mma.rst:67
> (Diff revision 1)
> >      "Focus Installed" -> true           // If Focus for Android is installed.
> >      "Klar Installed" -> true            // If Klar for Android is installed.
> >      "Pocket Installed" -> true          // If Pocket for Android is installed.
> >      "Signed In Sync" -> true            // If the user has signed in to Mozilla account.
> >      "Default Browser" -> true           // If the user has set Fennec as default browser.
> > +    "Pocket in Top Sites" -> true       // If Top Sites home panel contain websites recommended by Pocket
> 
> Nit: this sounds kind of passive, maybe something like "If Pocket
> recommendations are enabled (by default or through user action)"

Thanks Chenxia!
Because the initial change has already been merged I've made a new ticket for this - bug 1453276
Attachment #8966501 - Flags: review?(sdaswani)
Attachment #8966915 - Attachment is obsolete: false
Attachment #8966915 - Flags: review?(michael.l.comella)
Comment on attachment 8966915 [details]
Bug 1452173 - Add Leanplum documentation for Users Opting out of pocket in top sites;

This is redundant to bug 1453276.
Attachment #8966915 - Attachment is obsolete: true
Attachment #8966915 - Flags: review?(michael.l.comella)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: