mozilla / security-advisor-shield-study

Mozilla Public License 2.0
2 stars 7 forks source link

When users agree to sign up for sync, we should end the study immediately #59

Closed Osmose closed 7 years ago

Osmose commented 7 years ago

If we successfully convince a user to sign up for sync, instead of the current behavior of opening a tab to about:accounts, we should instead end the study immediately, and redirect them to the post-study-survey. We should also include a parameter in the survey URL indicating that the user ended the study early due to conversion, as opposed to ending the study after the 2-week duration.

Osmose commented 7 years ago

@rayborn @gregglind @MattGrimes I've worked a bit on implementing this, and I've found that some of the potential ways users can exit the study aren't logging consistently, so I want to confirm the list of possibilities with ya'll before applying fixes. Here's the list of end states I can think of, and what they should log (along with the study_name and branch, like how the study template logs currently) and whether we should survey them:

Does that sound right?

MattGrimes commented 7 years ago

That looks correct to me. @rayborn when we see reason=user-converted-feature-offer we need to show some special language in the survey explaining to the user that this feature isn't currently available. I don't think anything special is needed for reason=user-converted-no-offer.

ghost commented 7 years ago

Looks comprehensive to me. I'll start modifying the survey accordingly.

ghost commented 7 years ago

@MattGrimes - How does this sound:

MattGrimes commented 7 years ago

Here's my thoughs:

Survey with reason=user-converted-no-offer: I'd want to know why they decided to sign up for an account. Was it that we offered sync? Was it because they THOUGHT it had something to do with the password notification idea? Had they heard of accounts before?

Survey with reason=user-converted-feature-offer: Same as above. They signed up, we assume, because we promised that they would get password notifications. Would be good to validate that. Maybe a maxdiff sort of thing with our current account features + password notifications. Could throw vpn in there too if we wanted just for kicks.

Survey with reason=user-disabled-all-warnings: I want to know if they thought it was creepy. Are they worried about us knowing their passwords? Maybe they don't believe it's real?

ghost commented 7 years ago

Thanks for the ideas. I didn't add a maxdiff. I'm concerned about having a lot of questions and max diff is pretty time consuming. Instead I just have a general "Access to advanced services such as a VPN to keep your data private on public networks" as a checkbox option for Accounts expectations. I'm open to alternatives.

@MattGrimes - please review the following links:

MattGrimes commented 7 years ago

I put some comments in the surveys, but here's my feedback:

  1. I don't know if we even need to survey users that are inelligable. They get DQ'd the moment they install the add-on, so they have nothing to tell us.
  2. I'd like to find out if users think notification in the browser, notification by email, or both is the preferred way of being told about password breeches.
  3. Do users prefer to only be told about password issues on sites they have an account saved for. This test does not look at saved passwords.
  4. If you end the study or disable all the warnings, I'd still like the user to answer questions like "I believe mozilla has my best interest in mind" etc.
  5. For the user converted surveys, I'd also like them to answer the remainder of the questions. I assume (bold claim) that lots of people will end the study early by clicking on that sign up button. Even if they only saw one password notification I'd like to know if they thought it was creepy, helpful, etc.
ghost commented 7 years ago

I don't know if we even need to survey users that are inelligable. They get DQ'd the moment they install the add-on, so they have nothing to tell us.

Commenting on this particularly since this needs resolved ASAP since it may be a code change. I thought that this was for people who enabled sync mid experiment, not people immediately DQed. @MattGrimes, @Osmose - am I confused?

I'll address the other comments shortly.

Osmose commented 7 years ago

Mentioned this during the meeting, but @rayborn 's explanation is correct. Only users who enable sync mid-experiment are surveyed.

ghost commented 7 years ago

@MattGrimes - please review the following links:

==End of Study==

==Sync Converted==

==User Disabled==

Per Kamyar’s feedback

ghost commented 7 years ago

PS sorry for commenting on a closed issue, but the discussion started here, so I want to continue it here.

MattGrimes commented 7 years ago

Looks good to me. I think we're ready to launch tomorrow morning.