Skip to content

Commit 76174e8

Browse files
committed
Remove setting prefs for My Site Dashboard default tab experiment.
1 parent 8076e3f commit 76174e8

3 files changed

Lines changed: 0 additions & 31 deletions

File tree

WordPress/src/main/java/org/wordpress/android/ui/prefs/AppPrefs.java

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1386,17 +1386,6 @@ public static List<JetpackCapability> getSiteJetpackCapabilities(long remoteSite
13861386
return capabilities;
13871387
}
13881388

1389-
public static boolean isMySiteDefaultTabExperimentVariantAssigned() {
1390-
return getBoolean(
1391-
DeletablePrefKey.MY_SITE_DEFAULT_TAB_EXPERIMENT_VARIANT_ASSIGNED,
1392-
false
1393-
);
1394-
}
1395-
1396-
public static void setMySiteDefaultTabExperimentVariantAssigned() {
1397-
setBoolean(DeletablePrefKey.MY_SITE_DEFAULT_TAB_EXPERIMENT_VARIANT_ASSIGNED, true);
1398-
}
1399-
14001389
public static Date getSkippedPromptDay(int siteId) {
14011390
long promptSkippedMillis = prefs().getLong(getSkippedBloggingPromptDayConfigKey(siteId), 0);
14021391
if (promptSkippedMillis == 0) {
@@ -1417,16 +1406,6 @@ public static void setSkippedPromptDay(@Nullable Date date, int siteId) {
14171406
return DeletablePrefKey.SKIPPED_BLOGGING_PROMPT_DAY.name() + siteId;
14181407
}
14191408

1420-
public static void setInitialScreenFromMySiteDefaultTabExperimentVariant(String variant) {
1421-
// This supports the MySiteDefaultTab AB Experiment.
1422-
// AppSettings are undeletable across logouts and keys are all lower case.
1423-
// This method will be removed when the experiment has completed and thus
1424-
// the settings will be maintained only from the AppSettings view{
1425-
String initialScreen = variant.equals(MySiteTabType.SITE_MENU.getTrackingLabel())
1426-
? MySiteTabType.SITE_MENU.getLabel() : MySiteTabType.DASHBOARD.getLabel();
1427-
setString(UndeletablePrefKey.wp_pref_initial_screen, initialScreen);
1428-
}
1429-
14301409
public static String getMySiteInitialScreen() {
14311410
return getString(
14321411
UndeletablePrefKey.wp_pref_initial_screen,

WordPress/src/main/java/org/wordpress/android/ui/prefs/AppPrefsWrapper.kt

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -220,13 +220,6 @@ class AppPrefsWrapper @Inject constructor() {
220220
fun setLastSelectedQuickStartTypeForSite(quickStartType: QuickStartType, siteLocalId: Long) =
221221
AppPrefs.setLastSelectedQuickStartTypeForSite(quickStartType, siteLocalId)
222222

223-
fun isMySiteDefaultTabExperimentVariantAssigned() = AppPrefs.isMySiteDefaultTabExperimentVariantAssigned()
224-
225-
fun setMySiteDefaultTabExperimentVariantAssigned() = AppPrefs.setMySiteDefaultTabExperimentVariantAssigned()
226-
227-
fun setInitialScreenFromMySiteDefaultTabExperimentVariant(variant: String) =
228-
AppPrefs.setInitialScreenFromMySiteDefaultTabExperimentVariant(variant)
229-
230223
fun getMySiteInitialScreen(): String = AppPrefs.getMySiteInitialScreen()
231224

232225
fun setSkippedPromptDay(date: Date?, siteId: Int) = AppPrefs.setSkippedPromptDay(date, siteId)

WordPress/src/main/java/org/wordpress/android/ui/prefs/AppSettingsFragment.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@
4646
import org.wordpress.android.ui.about.UnifiedAboutActivity;
4747
import org.wordpress.android.ui.debug.DebugSettingsActivity;
4848
import org.wordpress.android.ui.mysite.jetpackbadge.JetpackPoweredBottomSheetFragment;
49-
import org.wordpress.android.ui.mysite.tabs.MySiteDefaultTabExperiment;
5049
import org.wordpress.android.ui.mysite.tabs.MySiteTabType;
5150
import org.wordpress.android.ui.prefs.language.LocalePickerBottomSheet;
5251
import org.wordpress.android.ui.prefs.language.LocalePickerBottomSheet.LocalePickerCallback;
@@ -105,7 +104,6 @@ public class AppSettingsFragment extends PreferenceFragment
105104
@Inject BuildConfigWrapper mBuildConfigWrapper;
106105
@Inject UnifiedAboutFeatureConfig mUnifiedAboutFeatureConfig;
107106
@Inject MySiteDashboardTabsFeatureConfig mMySiteDashboardTabsFeatureConfig;
108-
@Inject MySiteDefaultTabExperiment mMySiteDefaultTabExperiment;
109107
@Inject JetpackBrandingUtils mJetpackBrandingUtils;
110108

111109
private static final String TRACK_STYLE = "style";
@@ -474,7 +472,6 @@ public boolean onPreferenceChange(Preference preference, Object newValue) {
474472
Map<String, Object> properties = new HashMap<>();
475473
properties.put("selected", trackValue);
476474
AnalyticsTracker.track(Stat.APP_SETTINGS_INITIAL_SCREEN_CHANGED, properties);
477-
mMySiteDefaultTabExperiment.changeExperimentVariantAssignmentIfNeeded(trackValue);
478475
} else if (preference == mReportCrashPref) {
479476
AnalyticsTracker.track(Stat.PRIVACY_SETTINGS_REPORT_CRASHES_TOGGLED, Collections
480477
.singletonMap(TRACK_ENABLED, newValue));

0 commit comments

Comments
 (0)