Rename null to none when no ETP is enabled

mozilla-mobile · Jul 16, 2020 · 1db2335 · 1db2335
1 parent 1e06819
commit 1db2335
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/src/main/java/org/mozilla/fenix/components/metrics/LeanplumMetricsService.kt b/app/src/main/java/org/mozilla/fenix/components/metrics/LeanplumMetricsService.kt
@@ -104,7 +104,7 @@ class LeanplumMetricsService(private val application: Application) : MetricsServ
 
             val trackingProtection = application.settings().run {
                 when {
-                    !shouldUseTrackingProtection -> null
+                    !shouldUseTrackingProtection -> "none"
                     useStandardTrackingProtection -> "standard"
                     useStrictTrackingProtection -> "strict"
                     else -> "custom"

diff --git a/docs/mma.md b/docs/mma.md
@@ -151,7 +151,7 @@ User Attributes
   </tr>
   <tr>
     <td>`tracking_protection_setting`</td>
-    <td>A string indicating the level at which the user has set tracking protection. Possible values are `null`, `standard`, `strict` and `custom`</td>
+    <td>A string indicating the level at which the user has set tracking protection. Possible values are `none`, `standard`, `strict` and `custom`</td>
     <td><a href="https://github.com/mozilla-mobile/fenix/pull/11965#issuecomment-649731798">#11965</a></td>
   </tr>
   <tr>