Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add check-box for Firefox Account on the Monitor homepage #1665

Closed
winstonbowden opened this issue Apr 24, 2020 · 10 comments
Closed

Add check-box for Firefox Account on the Monitor homepage #1665

winstonbowden opened this issue Apr 24, 2020 · 10 comments
Assignees
Labels
growth-team Growth team

Comments

@winstonbowden
Copy link

winstonbowden commented Apr 24, 2020

Graduate Monitor Experiment 1 - Issue #1587

Screen Shot 2020-05-04 at 9 55 08 AM

@maxxcrawford maxxcrawford self-assigned this Apr 24, 2020
@maxxcrawford maxxcrawford added the growth-team Growth team label Apr 24, 2020
@maxxcrawford
Copy link
Contributor

  • Copy will need to be localized

@winstonbowden
Copy link
Author

EN only for now.

@changecourse
Copy link

@betsymi would you be willing to do a drive-by review of the string associated with the checkbox? This string ran for our experiment 1 already, and thus we didn't do a formal review, but since the data came back positively, we're wanting to do our due diligence now.

@betsymi
Copy link

betsymi commented Apr 29, 2020

@betsymi would you be willing to do a drive-by review of the string associated with the checkbox? This string ran for our experiment 1 already, and thus we didn't do a formal review, but since the data came back positively, we're wanting to do our due diligence now.

Sure, thanks for asking. Has legal had eyes on it as well? I think the string as it stands could be a little misleading, because we don't automatically notify when accounts get breached. We automatically notify when we find out about them, when they get added to the database.

Are you asking for me to comment my drive-by string, right here in this thread? Or do I need to put it somewhere else?

@changecourse
Copy link

Legal has approved it (correct @winstonbowden?)

Right here on this thread is perfect, and I can work with the team from there.

@winstonbowden
Copy link
Author

Thanks for asking about legal. They did have a request, which is call it out as a "known" breach. Something like this:
Stay Safe: get automated alerts when we know about an account breach.

@changecourse
Copy link

Updated string from @betsymi

Get email alerts when your info appears in a known breach

@winstonbowden let's discuss whether or not we need to re-run vs. moving forward with this. IMHO, I don't think we're at risk to limit impact with this string vs. our original experiment string, with regards to impact/growth.

@winstonbowden
Copy link
Author

winstonbowden commented May 1, 2020

@changecourse @betsymi thanks! I'm good with the proposed string as long as we keep "Stay safe" in front of it.

@changecourse
Copy link

changecourse commented May 4, 2020

"Final" string:
Stay safe: Get email alerts when your info appears in a known breach

Screen Shot 2020-05-04 at 9 55 08 AM

Ready for graduation, @winstonbowden @maxxcrawford

maxxcrawford added a commit that referenced this issue May 5, 2020
maxxcrawford added a commit that referenced this issue May 5, 2020
This reverts commit 8fe874f.

Removed white space changes
maxxcrawford added a commit that referenced this issue May 5, 2020
Revert "Fixed #1665 - Added string for homepage checkbox"

This reverts commit 8fe874f.

Removed white space changes
@maxxcrawford maxxcrawford reopened this May 5, 2020
@maxxcrawford
Copy link
Contributor

The string is now in master and will get localized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
growth-team Growth team
Projects
None yet
Development

No branches or pull requests

4 participants