Opened on 09/25/2015 at 10:19:42 AM
Closed on 09/25/2015 at 11:59:08 AM
#3125 closed change (fixed)
Improve the second Adblock Browser notification test
Reported by: | fhd | Assignee: | |
---|---|---|---|
Priority: | P2 | Milestone: | |
Module: | Sitescripts | Keywords: | |
Cc: | Blocked By: | #3124 | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by fhd)
Background
We have started the second notification test last night, see: https://codereview.adblockplus.org/29328597/
However, there are some problems. We had to reopen #2981, because it is still not solved, and thus the test sample is too large. We also forgot to consider testing the new normal notification type in Chrome (#3020).
What to change
Modify notification 2:
- Change severity to normal
- Change links for each variant: adblock_browser_promotion_1 should become adblock_browser_promotion_4, adblock_browser_promotion_2 should become adblock_browser_promotion_5 and adblock_browser_promotion_3 should become adblock_browser_promotion_6. (Otherwise we can't tell which were clicked from an notification of the previous type.)
- Change sample for each variant from 0.017 to 0.005. (This will put at least 1.5% of the users into one of the test groups - very likely more, because of #2981.)
Attachments (0)
Change History (8)
comment:5 Changed on 09/25/2015 at 10:31:19 AM by fhd
- Summary changed from Improve the second notification test to Improve the second Adblock Browser notification test
comment:6 Changed on 09/25/2015 at 10:32:26 AM by fhd
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:7 Changed on 09/25/2015 at 10:57:32 AM by fhd
comment:8 Changed on 09/25/2015 at 11:59:08 AM by fhd
- Resolution set to fixed
- Status changed from reviewing to closed
Reviewed now.
Note: See
TracTickets for help on using
tickets.
Had to push this pre-review due to time constraints: https://hg.adblockplus.org/notifications/rev/63bafa8fe4da