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

Additional "default browser" prompts: control #5486

Open
wants to merge 1 commit into
base: feature/lukasz-p/default-browser-prompts-variant-2
Choose a base branch
from

Conversation

LukasPaczos
Copy link
Contributor

Task/Issue URL: https://app.asana.com/0/72649045549333/1209176678727048/f

Description

Builds on top of #5476 and adds control.

Steps to test this PR

This PR relies on the new A/B/N framework for rollout, so to test you'll need a custom privacy config which includes the feature definition, for example:

"defaultBrowserPrompts": {
  "state": "enabled",
  "exceptions": [],
  "features": {
    "additionalPrompts": {
      "state": "enabled",
      "rollout": {
        "steps": [
          {
            "percent": 100
          }
        ]
      },
      "settings": {
        "activeDaysUntilStage1": 1,
        "activeDaysUntilStage2": 3,
        "activeDaysUntilStop": 5
      },
      "cohorts": [
        {
          "name": "control",
          "weight": 1
        },
        {
          "name": "variant_1",
          "weight": 0
        },
        {
          "name": "variant_2",
          "weight": 0
        },
        {
          "name": "variant_3",
          "weight": 0
        }
      ]
    }
  },
}

or you can apply this JSON Blob:

diff --git a/privacy-config/privacy-config-api/src/main/java/com/duckduckgo/privacy/config/api/PrivacyFeatureName.kt b/privacy-config/privacy-config-api/src/main/java/com/duckduckgo/privacy/config/api/PrivacyFeatureName.kt
index ca0870311..5a904f76b 100644
--- a/privacy-config/privacy-config-api/src/main/java/com/duckduckgo/privacy/config/api/PrivacyFeatureName.kt
+++ b/privacy-config/privacy-config-api/src/main/java/com/duckduckgo/privacy/config/api/PrivacyFeatureName.kt
@@ -27,4 +27,5 @@ enum class PrivacyFeatureName(val value: String) {
     TrackingParametersFeatureName("trackingParameters"),
 }
 
-const val PRIVACY_REMOTE_CONFIG_URL = "https://staticcdn.duckduckgo.com/trackerblocking/config/v4/android-config.json"
+// const val PRIVACY_REMOTE_CONFIG_URL = "https://staticcdn.duckduckgo.com/trackerblocking/config/v4/android-config.json"
+const val PRIVACY_REMOTE_CONFIG_URL = "https://www.jsonblob.com/api/1329816455226777600"

Once built with the right config, ensure that you don't have DDG app set as a default browser, only then you'll be assigned to the experiment.

As this is control, during the experiment, you should see no dialogs, no highlights, and no new menu items.

@LukasPaczos LukasPaczos requested a review from malmstein January 17, 2025 14:20
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-variant-2 branch from 042d38e to baed26a Compare January 20, 2025 12:10
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-control branch from 674f63c to 95c0002 Compare January 20, 2025 12:16
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-variant-2 branch from baed26a to 916a745 Compare January 22, 2025 12:25
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-control branch from 95c0002 to f0c21d0 Compare January 22, 2025 12:26
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-variant-2 branch from 916a745 to 4a97b9f Compare January 22, 2025 12:49
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-control branch from f0c21d0 to f01dbbf Compare January 22, 2025 12:49
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-variant-2 branch from 4a97b9f to 6c7a3cd Compare January 23, 2025 13:19
@LukasPaczos LukasPaczos force-pushed the feature/lukasz-p/default-browser-prompts-control branch from f01dbbf to 3c1b5da Compare January 23, 2025 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant