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

GH-2159: Update frequency reduction, AB test opt-out, remove Cliqz AB test #608

Merged
merged 24 commits into from Sep 24, 2020
Merged
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
d5ea47a
Reduce scheduledTasks and autoUpdatedBugDb frequencies from once an h…
wlycdgr Sep 21, 2020
544043c
Refactor autoUpdateBugDb so it always does the same thing when called…
wlycdgr Sep 21, 2020
9f2cf6c
Update autoUpdateBugDB documentation and refactor a little more to ma…
wlycdgr Sep 21, 2020
568acea
Clean up autoUpdateBugDB a bit more: Clarify docs, flatten, and move …
wlycdgr Sep 21, 2020
5c624b3
Minor comment in autoUpdateBugDb
wlycdgr Sep 21, 2020
ef45c82
Renamed autoUpdateBugDb to clarify connection to checkLibraryVersion.…
wlycdgr Sep 21, 2020
ad20931
Edit db update function names and docs to improve clarity
wlycdgr Sep 21, 2020
c761b7c
Remove Cliqz anti-tracking A/B test config code from background. Tidy…
wlycdgr Sep 21, 2020
b0a54c5
Split up and reorganize setupABTest so outside code can only call tho…
wlycdgr Sep 21, 2020
94cfe13
Merge in develop
wlycdgr Sep 22, 2020
c563a1d
Clean up OptIn a bit in preparation for adding A/B test checkbox
wlycdgr Sep 22, 2020
89d6e0f
Finish OptIn tidy-up
wlycdgr Sep 22, 2020
2572a2d
Stub out new A/B test opt-in setting. Add draft strings for it.
wlycdgr Sep 22, 2020
2d42906
Implement the enable_abtests conf setting and plug it into the checkb…
wlycdgr Sep 23, 2020
1f3a03f
Do not call setupABTests if enable_abtests is false
wlycdgr Sep 23, 2020
cba107f
Skip the call to the AB server altogether if ab tests are disabled
wlycdgr Sep 23, 2020
f369ad2
Update AB test setting tooltip copy. remove enable_abtests from unins…
wlycdgr Sep 23, 2020
a0f0f0c
Remove unused import from Metrics
wlycdgr Sep 23, 2020
fa54859
Update test snapshot
wlycdgr Sep 23, 2020
004506b
Fix typo
wlycdgr Sep 23, 2020
98644ff
Fix comment errors
wlycdgr Sep 23, 2020
9f6b99d
Remove bloom filter Cliqz test
wlycdgr Sep 24, 2020
f1b4835
remove comment
christophertino Sep 24, 2020
6d8d250
Merge branch 'develop' into GH-2159
christophertino Sep 24, 2020
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Stub out new A/B test opt-in setting. Add draft strings for it.

  • Loading branch information
wlycdgr committed Sep 22, 2020
commit 2572a2de3370a312877e8d76cf786a03a1688958
@@ -947,6 +947,9 @@
"settings_allow_offers": {
"message": "Participating in Ghostery Rewards"
},
"settings_allow_abtests": {
"message": "Participating in A/B Tests"
},
"settings_signin_create_header": {
"message": "Sign In / Create Account"
},
@@ -1037,6 +1040,9 @@
"settings_offers_tooltip": {
"message": "Ghostery Rewards is a private-by-design feature that delivers you discounts and special offers from our partner companies as you browse."
},
"settings_abtests_tooltip": {
"message": "Participating in A/B tests helps Ghostery understand which version of a new layout or feature users like you prefer. The testing mechanism uses a random number generated on install to determine which version of a beta feature you see."
},
"settings_opt_in": {
"message": "Opt In / Out"
},
@@ -85,6 +85,12 @@ const OptIn = ({ settingsData, toggleCheckbox }) => {
tooltipSVG(t('settings_offers_tooltip'), 'up'),
'offers-section'
)}
{!IS_CLIQZ && option(
checkbox('allow-abtests', true),
labelFor('allow-abtests', t('settings_allow_abtests')),
tooltipSVG(t('settings_abtests_tooltip'), 'up'),
'abtests-section'
)}
</div>
</div>
</div>
ProTip! Use n and p to navigate between commits in a pull request.