NT-1989: Removed any code related with "Email Verification" feature f…

…lag, no related code on the app for "Go Rewardless", "Native Checkout", and "Creator View" (#1302)
kickstarter · Jun 23, 2021 · 5d909a8 · 5d909a8
1 parent 71e72ce
commit 5d909a8
Show file tree

Hide file tree

Showing 15 changed files with 31 additions and 543 deletions.
diff --git a/app/src/internal/java/com/kickstarter/ui/activities/FeatureFlagsActivity.kt b/app/src/internal/java/com/kickstarter/ui/activities/FeatureFlagsActivity.kt
@@ -11,7 +11,7 @@ import com.kickstarter.libs.preferences.StringPreferenceType
 import com.kickstarter.libs.qualifiers.RequiresActivityViewModel
 import com.kickstarter.libs.rx.transformers.Transformers.observeForUI
 import com.kickstarter.libs.utils.BooleanUtils
-import com.kickstarter.libs.utils.ConfigFeatureName.SEGMENT_ENABLED
+import com.kickstarter.libs.utils.ConfigFeatureFlagName.SEGMENT_ENABLED
 import com.kickstarter.ui.adapters.FeatureFlagsAdapter
 import com.kickstarter.ui.itemdecorations.TableItemDecoration
 import com.kickstarter.ui.viewholders.FeatureFlagViewHolder
@@ -67,7 +67,7 @@ class FeatureFlagsActivity : BaseActivity<FeatureFlagsViewModel.ViewModel>(), Fe
     override fun featureOptionToggle(featureName: String, isEnabled: Boolean) {
 
         when (featureName) {
-            SEGMENT_ENABLED.configFeatureName -> {
+            SEGMENT_ENABLED.featureFlag -> {
                 this.viewModel.inputs.updateSegmentFlag(isEnabled, featuresFlagPreference)
             }
         }

diff --git a/app/src/internal/java/com/kickstarter/ui/activities/InternalToolsActivity.kt b/app/src/internal/java/com/kickstarter/ui/activities/InternalToolsActivity.kt
@@ -7,7 +7,6 @@ import android.content.DialogInterface
 import android.content.Intent
 import android.content.IntentFilter
 import android.os.Bundle
-import android.util.Log
 import android.view.View
 import android.webkit.URLUtil
 import android.widget.EditText
@@ -29,13 +28,11 @@ import com.kickstarter.libs.Logout
 import com.kickstarter.libs.preferences.StringPreferenceType
 import com.kickstarter.libs.qualifiers.ApiEndpointPreference
 import com.kickstarter.libs.qualifiers.RequiresActivityViewModel
-import com.kickstarter.libs.utils.LoginHelper
 import com.kickstarter.libs.utils.Secrets
 import com.kickstarter.libs.utils.TransitionUtils
 import com.kickstarter.libs.utils.ViewUtils
 import com.kickstarter.libs.utils.WorkUtils
 import com.kickstarter.services.firebase.ResetDeviceIdWorker
-import com.kickstarter.ui.fragments.Callbacks
 import com.kickstarter.viewmodels.InternalToolsViewModel
 import org.joda.time.format.DateTimeFormat
 import java.util.concurrent.TimeUnit
@@ -107,10 +104,6 @@ class InternalToolsActivity : BaseActivity<InternalToolsViewModel>() {
             featureFlagsClick()
         }
 
-        binding.emailVerificationButton.setOnClickListener {
-            emailVerificationInterstitialClick()
-        }
-
         binding.resetDeviceId.setOnClickListener {
             resetDeviceIdClick()
         }
@@ -164,18 +157,6 @@ class InternalToolsActivity : BaseActivity<InternalToolsViewModel>() {
         startActivity(featureFlagIntent)
     }
 
-    private fun emailVerificationInterstitialClick() {
-        LoginHelper.showInterstitialFragment(
-            this.supportFragmentManager,
-            R.id.email_verification_interstitial_fragment_container,
-            object : Callbacks {
-                override fun onDismiss() {
-                    Log.i(this@InternalToolsActivity.localClassName, "Placeholder for callback function")
-                }
-            }
-        )
-    }
-
     private fun resetDeviceIdClick() {
         showDeviceId(false)
         val request: OneTimeWorkRequest = OneTimeWorkRequest.Builder(ResetDeviceIdWorker::class.java)

diff --git a/app/src/internal/java/com/kickstarter/viewmodels/FeatureFlagsViewModel.kt b/app/src/internal/java/com/kickstarter/viewmodels/FeatureFlagsViewModel.kt
@@ -8,7 +8,7 @@ import com.kickstarter.libs.CurrentUserType
 import com.kickstarter.libs.Environment
 import com.kickstarter.libs.ExperimentsClientType
 import com.kickstarter.libs.preferences.StringPreferenceType
-import com.kickstarter.libs.utils.ConfigFeatureName.SEGMENT_ENABLED
+import com.kickstarter.libs.utils.ConfigFeatureFlagName.SEGMENT_ENABLED
 import com.kickstarter.libs.utils.extensions.setUserFeatureFlagsPrefWithFeatureFlag
 import com.kickstarter.model.FeatureFlagsModel
 import com.kickstarter.ui.activities.FeatureFlagsActivity
@@ -57,7 +57,7 @@ interface FeatureFlagsViewModel {
                 .map { it?.entries?.toList() ?: listOf<Map.Entry<String, Boolean>>() }
                 .map { it.filter { entry -> entry.key.startsWith("android_") } }
                 .map { it.sortedBy { entry -> entry.key } }
-                .map { it.map { entry -> FeatureFlagsModel(entry.key, entry.value, entry.key.equals(SEGMENT_ENABLED.configFeatureName)) }.toList() }
+                .map { it.map { entry -> FeatureFlagsModel(entry.key, entry.value, entry.key.equals(SEGMENT_ENABLED.featureFlag)) }.toList() }
                 .compose(bindToLifecycle())
                 .subscribe(this.configFeatures)
 
@@ -73,7 +73,7 @@ interface FeatureFlagsViewModel {
                 .subscribe {
                     config?.setUserFeatureFlagsPrefWithFeatureFlag(
                         it.second,
-                        SEGMENT_ENABLED.configFeatureName,
+                        SEGMENT_ENABLED.featureFlag,
                         it.first
                     )
                     environment.currentConfig().config(config)

diff --git a/app/src/internal/res/layout/activity_feature_flags.xml b/app/src/internal/res/layout/activity_feature_flags.xml
@@ -3,7 +3,7 @@
   xmlns:app="http://schemas.android.com/apk/res-auto"
   xmlns:tools="http://schemas.android.com/tools"
   android:layout_width="match_parent"
-  android:layout_height="wrap_content">
+  android:layout_height="match_parent">
 
   <com.google.android.material.appbar.AppBarLayout
     android:layout_width="match_parent"
@@ -68,15 +68,6 @@
         tools:listitem="@layout/item_feature_flag" />
 
       <include layout="@layout/horizontal_line_1dp_view" />
-
-      <TextView
-        style="@style/CalloutSecondaryMedium"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_marginTop="@dimen/grid_3"
-        android:paddingStart="@dimen/activity_vertical_margin"
-        android:paddingEnd="@dimen/activity_vertical_margin"
-        android:text="@string/Overrides" />
     </LinearLayout>
 
   </ScrollView>

diff --git a/app/src/internal/res/layout/internal_tools_layout.xml b/app/src/internal/res/layout/internal_tools_layout.xml
@@ -246,23 +246,8 @@
         android:text="@string/Force_a_crash" />
 
       <include layout="@layout/horizontal_line_1dp_view" />
-
-      <Button
-          android:id="@+id/email_verification_button"
-          style="@style/Button"
-          app:background="@color/green_alpha_6"
-          android:layout_width="match_parent"
-          android:layout_marginTop="@dimen/grid_1"
-          android:layout_marginBottom="@dimen/grid_1"
-          android:text="@string/email_verification_interstitial" />
-
     </LinearLayout>
 
   </ScrollView>
 
-  <FrameLayout
-      android:id="@+id/email_verification_interstitial_fragment_container"
-      android:layout_width="match_parent"
-      android:layout_height="match_parent"/>
-
 </androidx.coordinatorlayout.widget.CoordinatorLayout>
diff --git a/app/src/main/java/com/kickstarter/libs/TrackingClient.kt b/app/src/main/java/com/kickstarter/libs/TrackingClient.kt
@@ -12,7 +12,7 @@ import com.google.firebase.iid.FirebaseInstanceId
 import com.kickstarter.BuildConfig
 import com.kickstarter.R
 import com.kickstarter.libs.qualifiers.ApplicationContext
-import com.kickstarter.libs.utils.ConfigFeatureName.SEGMENT_ENABLED
+import com.kickstarter.libs.utils.ConfigFeatureFlagName.SEGMENT_ENABLED
 import com.kickstarter.libs.utils.WebUtils
 import com.kickstarter.libs.utils.extensions.currentVariants
 import com.kickstarter.libs.utils.extensions.enabledFeatureFlags
@@ -60,7 +60,7 @@ abstract class TrackingClient(
 
     override fun isEnabled(): Boolean {
         return if (type() == Type.SEGMENT) {
-            this.config?.isFeatureFlagEnabled(SEGMENT_ENABLED.configFeatureName) ?: false
+            this.config?.isFeatureFlagEnabled(SEGMENT_ENABLED.featureFlag) ?: false
         } else true
     }
 

diff --git a/app/src/main/java/com/kickstarter/libs/utils/ConfigFeatureFlagName.kt b/app/src/main/java/com/kickstarter/libs/utils/ConfigFeatureFlagName.kt
@@ -0,0 +1,10 @@
+package com.kickstarter.libs.utils
+
+enum class ConfigFeatureFlagName(val featureFlag: String) {
+
+    /**
+     * Internal name for "android_segment"
+     * @{link https://github.com/kickstarter/kickstarter/blob/e9c61ea9f1e4817bb64560db7c32f3b9704cdc60/config/features.yml#L219}
+     */
+    SEGMENT_ENABLED("android_segment")
+}
diff --git a/app/src/main/java/com/kickstarter/libs/utils/ConfigFeatureName.kt b/app/src/main/java/com/kickstarter/libs/utils/ConfigFeatureName.kt
diff --git a/app/src/main/java/com/kickstarter/libs/utils/LoginHelper.kt b/app/src/main/java/com/kickstarter/libs/utils/LoginHelper.kt
diff --git a/app/src/main/java/com/kickstarter/libs/utils/extensions/ConfigExtension.kt b/app/src/main/java/com/kickstarter/libs/utils/extensions/ConfigExtension.kt
@@ -5,7 +5,7 @@ import com.google.gson.Gson
 import com.google.gson.reflect.TypeToken
 import com.kickstarter.libs.Config
 import com.kickstarter.libs.preferences.StringPreferenceType
-import com.kickstarter.libs.utils.ConfigFeatureName
+import com.kickstarter.libs.utils.ConfigFeatureFlagName
 import org.json.JSONArray
 
 /**
@@ -68,8 +68,8 @@ fun Config.syncUserFeatureFlagsFromPref(featuresFlagPreference: StringPreference
         featuresFlagPreference.get(), object : TypeToken<HashMap<String?, Boolean?>>() {}.type
     )
 
-    featuresFlagsMap[ConfigFeatureName.SEGMENT_ENABLED.configFeatureName]?.let {
-        this.features()?.put(ConfigFeatureName.SEGMENT_ENABLED.configFeatureName, it)
+    featuresFlagsMap[ConfigFeatureFlagName.SEGMENT_ENABLED.featureFlag]?.let {
+        this.features()?.put(ConfigFeatureFlagName.SEGMENT_ENABLED.featureFlag, it)
     }
 }
 

diff --git a/app/src/main/java/com/kickstarter/ui/fragments/EmailVerificationInterstitialFragment.kt b/app/src/main/java/com/kickstarter/ui/fragments/EmailVerificationInterstitialFragment.kt