Save to Pocket button sign up buttons test
Categories
(Firefox :: Pocket, enhancement, P1)
Tracking
()
People
(Reporter: thecount, Assigned: thecount)
Details
Attachments
(1 file)
|
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
We want to test a single signup button test.
We'll want a variant attached to this pref "extensions.pocket.loggedOutVariant"
For 1 button, and for a control.
We want to send the variant names to the utm params.
| Assignee | ||
Comment 1•5 years ago
|
||
[Tracking Requested - why for this release]:
| Assignee | ||
Comment 2•5 years ago
|
||
Updated•5 years ago
|
| Assignee | ||
Comment 3•5 years ago
•
|
||
To test:
- Log out of Pocket.
- Set extensions.pocket.loggedOutVariant to button_control
- Click the save to Pocket button on a page.
- Click Sign up with Firefox or Sign up with email.
expected: should see in the url these params for both buttons "utm_source=button_control" and "utm_campaign=logged_out_button_test"
- Change extensions.pocket.loggedOutVariant to button_variant
- Click the save to Pocket button on a page.
- Click Sign up
Expected: Should only have 1 button option, Sign up.
Expected: the url params should be "utm_source=button_variant" and "utm_campaign=logged_out_button_test"
Expected: Height of the panel should reasonably wrap the content.
Comment 5•5 years ago
|
||
| bugherder | ||
| Assignee | ||
Comment 6•5 years ago
•
|
||
Comment on attachment 9201520 [details]
Bug 1691126 - Save to Pocket button test.
Beta/Release Uplift Approval Request
- User impact if declined: We want to run an experiment with the save to Pocket buttons, 1 button vs 2.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce:
- Log out of Pocket.
- Set extensions.pocket.loggedOutVariant to button_control
- Click the save to Pocket button on a page.
- Click Sign up with Firefox or Sign up with email.
expected: should see in the url these params for both buttons "utm_source=button_control" and "utm_campaign=logged_out_button_test"
- Change extensions.pocket.loggedOutVariant to button_variant
- Click the save to Pocket button on a page.
- Click Sign up
Expected: Should only have 1 button option, Sign up.
Expected: the url params should be "utm_source=button_variant" and "utm_campaign=logged_out_button_test"
Expected: Height of the panel should reasonably wrap the content.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It's a simple change and behind a pref
- String changes made/needed: None
| Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 7•5 years ago
•
|
||
I have verified this enhancement and I can confirm the following :
-
The
set extensions.pocket.loggedOutVariantpref set tobutton_control:- The "utm_source=button_control" and "utm_campaign=logged_out_button_test" params are successfully displayed inside the URL.
-
The
set extensions.pocket.loggedOutVariantpref set tobutton_variant:- Only one button is displayed in the "Saved to Pocket" doorhanger, more exactly the "Sign up" one.
- The "utm_source=button_variant" and "utm_campaign=logged_out_button_test" params are successfully displayed inside the URL.
- All the elements are correctly rendered inside the "Saved to Pocket" doorhanger.
Verified using the latest Firefox Nightly (87.0a1 Build ID - 20210208092848) installed on Windows 10 x64, Mac 10.15.7, and Linux Mint 20 x64.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 8•5 years ago
|
||
Comment on attachment 9201520 [details]
Bug 1691126 - Save to Pocket button test.
Approved for 86 beta 8, thanks.
Comment 9•5 years ago
|
||
| bugherder uplift | ||
Comment 10•5 years ago
|
||
I have verified this enhancement and I can confirm the following :
-
The
set extensions.pocket.loggedOutVariantpref set tobutton_control:- The "utm_source=button_control" and "utm_campaign=logged_out_button_test" params are successfully displayed inside the URL.
-
The
set extensions.pocket.loggedOutVariantpref set tobutton_variant:- Only one button is displayed in the "Saved to Pocket" doorhanger, more exactly the "Sign up" one.
- The "utm_source=button_variant" and "utm_campaign=logged_out_button_test" params are successfully displayed inside the URL.
- All the elements are correctly rendered inside the "Saved to Pocket" doorhanger.
Verified using the latest Firefox Beta (86.0b8 Build ID - 20210209185733) installed on Windows 10 x64, Mac 10.15.7, and Linux Mint 20 x64.
Description
•