Closed
Bug 1072637
Opened 10 years ago
Closed 10 years ago
Allow opt in / opt out
Categories
(Marketplace Graveyard :: Integration, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
2014-11-25
People
(Reporter: andy+bugzilla, Assigned: robhudson, Mentored)
References
Details
Attachments
(1 file)
26.62 KB,
image/png
|
Details |
"As a user, I should be able to choose whether Marketplace gives me Personal recommendations and be able to opt-in/opt-out of "Personal" recommendations. The Default should be Opt-Out."
We'll need to:
* show this status to a user when they are looking at recommendations
* have a place to alter this (settings maybe?)
* affect the recommendations API call sent to the server, if opt-out is selected
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → dbialer
Priority: -- → P2
Comment 1•10 years ago
|
||
Per PRD, information is documented and recommendation is to default to have recommendations for those who are registered and logged in, not to have any personalized recommendations for those who choose not to be logged in or anonymous.
Once the UI is available for review, I'll confirm which way it should be implemented (default on or default off) and update the appropriate terms and privacy language.
Comment 3•10 years ago
|
||
(In reply to Andy McKay [:andym] from comment #0)
> * show this status to a user when they are looking at recommendations
What sort of items can be recommended? An app? A collection? Depending on the page where these items are seen, we might be able to generalise the recommendation signal. Maybe make it a tag or a flag.
> * have a place to alter this (settings maybe?)
Yes. Settings seems to be a good place to put this. We don’t need a different tab for it, either. Just a section called “Personalisation”. Under it, we can say something like “Yes. I would like to receive recommended apps and collections.”, and “No. I don’t want to receive them.”
Comment 4•10 years ago
|
||
CC Phil for some ideas on how to visually represent the “Recommended” flag. Of course, knowing where these recommended items live is important. It’ll allow us to design a signal that’s page appropriate.
Reporter | ||
Comment 5•10 years ago
|
||
https://marketplace-dev.allizom.org/recommended
There will be a "recommended" link next to "popular", its just turned off at the moment.
Reporter | ||
Comment 6•10 years ago
|
||
dbialer: sounds like opt in is the default
pwalm: sounds like bram had some ideas in comment 3 and comment 4, anything else on this?
...otherwise we'll get rob working on it.
Assignee: dbialer → pwalmsley
Comment 7•10 years ago
|
||
Reporter | ||
Updated•10 years ago
|
Assignee: pwalmsley → robhudson.mozbugs
Assignee | ||
Comment 8•10 years ago
|
||
We need to be sensitive to the performance and caching this could have on where we store this setting and how we send it to the front-end. Talking with Mat this was proposed:
* Hide the recommended tab by default. Anonymous users and opted-out users will not see it.
* Use the consumer_info API to pass the opt-out setting to the front-end. Once the consumer_info API loads and the user has not opted-out we show the recommended tab.
Assignee | ||
Updated•10 years ago
|
Status: NEW → ASSIGNED
Reporter | ||
Updated•10 years ago
|
Comment 9•10 years ago
|
||
FWIW, bug 993562 removed the settings link from the footer at desktop width. So anonymous users won't be able to see the settings page to opt out of Recommendations. But from comment 8 it appears anonymous users won't be seeing Recommendations anyway. Correct me if I'm wrong.
Assignee | ||
Comment 10•10 years ago
|
||
(In reply to Christopher Van Wiemeersch [:cvan] from comment #9)
> FWIW, bug 993562 removed the settings link from the footer at desktop width.
> So anonymous users won't be able to see the settings page to opt out of
> Recommendations. But from comment 8 it appears anonymous users won't be
> seeing Recommendations anyway. Correct me if I'm wrong.
Yes, anonymous users won't see recommendations. They also won't see the checkbox for the opt-out even if they go to the settings page.
Assignee | ||
Comment 11•10 years ago
|
||
https://github.com/mozilla/zamboni/commit/932666b
https://github.com/mozilla/fireplace/commit/192606e
The above commits simply add a user setting to enable or disable app recommendations. They currently don't show/hide the navigation or remove users from the daily dumps which will be handled by other bugs.
To verify:
* Ensure a logged out user does not see the checkbox at /settings.
* Ensure a logged in user does see the checkbox to enable/disable app recommendations.
* Ensure changing this setting persists between browsers or user sessions.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 12•10 years ago
|
||
Hi, I just talked with David B. and our suggested copy for the text is:
Replace "Personlisation" with "Recommendations"
"Yes, show me recommendations based on my installed apps. Learn More."
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 13•10 years ago
|
||
(In reply to Mika from comment #12)
> Hi, I just talked with David B. and our suggested copy for the text is:
>
> Replace "Personlisation" with "Recommendations"
>
> "Yes, show me recommendations based on my installed apps. Learn More."
Happy to change it.
I also saw that in the newsletter PDF Liz sent out it was requested we change this to something else ("I want to receive app recommendations from the Firefox Marketplace"). Can we all agree on what this text should be to avoid changing it many times which affects the localization team?
We don't currently have a destination for the "Learn More" link so I haven't added it yet.
Comment 14•10 years ago
|
||
For the Learn More link - David B has ideas on where this would be (marketplace page with static copy explaining recommendations). Alternatively it can link to the bit in the privacy notice that I'll write on how user data is used for recommendations.
What's the newsletter PDF? Happy to review and help decide final language there. For the opt-out page, the final language is as I wrote in comment 12.
Comment 15•10 years ago
|
||
(In reply to Mika from comment #14)
> What's the newsletter PDF? Happy to review and help decide final language there.
http://people.mozilla.org/~ehunt/newsletter/newsletter_v1.5.pdf
Comment 16•10 years ago
|
||
(In reply to Christopher Van Wiemeersch [:cvan] from comment #15)
> (In reply to Mika from comment #14)
> > What's the newsletter PDF? Happy to review and help decide final language there.
>
> http://people.mozilla.org/~ehunt/newsletter/newsletter_v1.5.pdf
The Account Setting mockup on page 7 of the pdf is different from the mockup attached in this bug. Which is the most current version of the Account Settings UI?
Comment 17•10 years ago
|
||
(In reply to Mika from comment #16)
> The Account Setting mockup on page 7 of the pdf is different from the mockup
> attached in this bug. Which is the most current version of the Account
> Settings UI?
Bram's attachment 8513236 [details] is from a few weeks ago. The PDF contains Elizabeth's proposed changes (as of November 14) for the Account Settings UI.
Comment 18•10 years ago
|
||
I was looking at an older version of the language, and didn't see the language in this bug before I made the latest PDF. Go ahead and keep the language you have.
I included the recommendations checkbox in the Newsletter PDF b/c I wanted to work out a system for that page that can grow to accommodate new things added over time. I'm not sure on the timing of this bug or if it will sync with the Newsletter one (1097152). If you want to move forward with the UI as you have it, that's ok by me. Then the Newsletter bug will adjust the UI when it happens.
Does that make sense? Happy to talk any time.
Comment 19•10 years ago
|
||
FWIW: You can sign in to https://marketplace-dev.allizom.org/settings and see the changes Rob Hudson landed on Tuesday on -dev:
https://www.dropbox.com/s/o96kypv4n9cee2o/Screenshot%202014-11-14%2017.04.34.png?dl=0
Assignee | ||
Comment 20•10 years ago
|
||
Text updated in commit:
https://github.com/mozilla/fireplace/commit/1b2e078edda1dec12f66ac1a04267485325e74a6
Assignee | ||
Comment 21•10 years ago
|
||
Since bug 1088304 is now for the "Learn more" link I'll close this as fixed.
Status: REOPENED → RESOLVED
Closed: 10 years ago → 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2014-11-25
Comment 22•10 years ago
|
||
Verified in FF36(Win7) in marketplace-dev.allizom.org
Logged out users do not see the recommended tab. A logged in user does see the checkbox to enable/disable app recommendations. The changes made are preserved between browsers or user sessions.
I have some issues/question:
1. For a new created user the recommendation is by default checked, is this intended?
2. The changes are preserved between sessions (same user has the recommendations on or off on any active sessions) but the apps listed in different sessions are different. Please view this screencast: http://screencast.com/t/9hCnoLlCyK
3. While logged out the user can still access https://marketplace-dev.allizom.org/recommended Screencast: http://screencast.com/t/c6LnlMumKP
Rob what do you think about the above issues?
Flags: needinfo?(robhudson.mozbugs)
Assignee | ||
Comment 23•10 years ago
|
||
(In reply to Madalin Cotetiu from comment #22)
> 1. For a new created user the recommendation is by default checked, is this
> intended?
Yes, we are defaulting to opt-in for this feature.
> 2. The changes are preserved between sessions (same user has the
> recommendations on or off on any active sessions) but the apps listed in
> different sessions are different. Please view this screencast:
> http://screencast.com/t/9hCnoLlCyK
That's fine. Each request to the recommendation system will refresh about 30% of the apps in the list.
> 3. While logged out the user can still access
> https://marketplace-dev.allizom.org/recommended Screencast:
> http://screencast.com/t/c6LnlMumKP
It's written this way, yes. I didn't do anything to block users if they know the URL I simply add/remove the navigation tab.
On the backend, if the user is opted out or logged out and hits this page we would recommend popular apps. But perhaps this can be confusing and we should either 404 or add some messaging like "You are currently logged out, log in to see personalized recommendations" or "You are currently opted out of recommendations, enable them in settings to see personalized recommendations" ?
Flags: needinfo?(robhudson.mozbugs)
Comment 24•10 years ago
|
||
Displaying popular apps to logged out user makes sense, but it also could be confusing. In my opinion recommendation are personal to each individual user and we should display them accordingly. A 404 page could also be confusing so, in my opinion, a message like "You are currently logged out, log in to see personalized recommendations" would be more appropriate.
Assignee | ||
Comment 25•10 years ago
|
||
I filed bug 1104865 for the user messaging.
Comment 26•10 years ago
|
||
As the original issue is fixed I will close this bug. We will continue to track the logged out user messaging in bug 1104865
Closing bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•