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

[MBL-696] Cleanup Optimizely Experiments #1812

Merged
merged 32 commits into from
Apr 19, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
738a49f
remove nativeOnboarding experiment
scottkicks Apr 11, 2023
f8f943b
remove onboardingCategoryPersonalizationFlow experiment
scottkicks Apr 11, 2023
0adadf4
remove nativeProjectCards experiment
scottkicks Apr 11, 2023
a85ebe3
remove nativeRiskMessaging experiment. leave affected code for now un…
scottkicks Apr 12, 2023
b979933
remove OptimizelyExperiment file
scottkicks Apr 12, 2023
9cf5163
remove goToLandingPage flow and LandingPage files
scottkicks Apr 12, 2023
aa6fbba
remove hasSeenLandingPage from KeyValueStoreType
scottkicks Apr 17, 2023
fe2743a
remove hasSeenCategoryPersonalizationFlow from KeyValueStoreType
scottkicks Apr 17, 2023
d2ef0b1
remove shouldSeeCategoryPersonalization(), goToCategoryPersonalizatio…
scottkicks Apr 17, 2023
1950b3b
rename testDeeplink and testOpenPushNotification_WhenOnboardingFlowIs…
scottkicks Apr 17, 2023
31ef1f0
add _Success suffix to testContinueUserActivity
scottkicks Apr 17, 2023
91ebb47
add _Success suffix to testPerformShortcutItem_
scottkicks Apr 17, 2023
b2fb981
remove experiment name from testLaunchShortcutItem_WhenOnboardingFlow…
scottkicks Apr 17, 2023
e1de13f
remove testEmailDeepLinking_WhenOnboardingFlowIsNotActive
scottkicks Apr 17, 2023
47f82e3
clean up projectsLoaded output in DiscoveryPageViewModel
scottkicks Apr 17, 2023
68bfcd3
.zero format suggestion
scottkicks Apr 17, 2023
771b6b5
put testConfigureDataSourceOptimizelyConfiguration test back
scottkicks Apr 17, 2023
415bb23
put testProjectsLoaded_Success back
scottkicks Apr 17, 2023
ecd1a95
put testContentInset_Success back
scottkicks Apr 17, 2023
a23f0e1
put testShowPersonalization_LoggedOut and testShowPersonalization_Log…
scottkicks Apr 17, 2023
82304a0
put testShowPersonalization_When_HasCompletedCategorySelection_IsFals…
scottkicks Apr 17, 2023
f2517e8
put testShowPersonalization_When_HasDismissedPersonalizationCell_IsTr…
scottkicks Apr 17, 2023
17e1e4e
remove experiments property and related methods from MockOptimizelyCl…
scottkicks Apr 17, 2023
b34e2d0
remove _Control suffix from testDiscoveryPageBackgroundColor
scottkicks Apr 17, 2023
2535c6b
clean up experiments related code from MockOptimizelyClient
scottkicks Apr 17, 2023
a4ced6f
Merge branch 'main' into scott/clean-up-experiments
msadoon Apr 17, 2023
2c48644
remove LandingViewModel and LandingViewModelTests
scottkicks Apr 18, 2023
71eab2f
Merge branch 'scott/clean-up-experiments' of https://github.com/kicks…
scottkicks Apr 18, 2023
696d888
remove unecessary test
scottkicks Apr 18, 2023
3f927b2
update testDiscoveryPageBackgroundColor test name to testIsNativeRisk…
scottkicks Apr 18, 2023
fd5bfda
undo naming since there is already a test with that name and use case
scottkicks Apr 18, 2023
2ca694e
remov LandingPageViewControllerTests and LandingViewControllerTests s…
scottkicks Apr 18, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 0 additions & 11 deletions Library/KeyValueStoreType.swift
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@ public enum AppKeys: String {
case deniedNotificationContexts = "com.kickstarter.KeyValueStoreType.deniedNotificationContexts"
case favoriteCategoryIds = "favorite_category_ids"
case hasCompletedCategoryPersonalizationFlow = "com.kickstarter.KeyValueStoreType.hasCompletedCategoryPersonalizationFlow"
case hasSeenCategoryPersonalizationFlow = "com.kickstarter.KeyValueStoreType.hasSeenCategoryPersonalizationFlow"
case hasDismissedPersonalizationCard = "com.kickstarter.KeyValueStoreType.hasDismissedPersonalizationCard"
case hasSeenFavoriteCategoryAlert = "com.kickstarter.KeyValueStoreType.hasSeenFavoriteCategoryAlert"
case hasSeenSaveProjectAlert = "com.kickstarter.KeyValueStoreType.hasSeenSaveProjectAlert"
Expand Down Expand Up @@ -40,7 +39,6 @@ public protocol KeyValueStoreType: AnyObject {
var hasCompletedCategoryPersonalizationFlow: Bool { get set }
var hasDismissedPersonalizationCard: Bool { get set }
var hasSeenAppRating: Bool { get set }
var hasSeenCategoryPersonalizationFlow: Bool { get set }
var hasSeenFavoriteCategoryAlert: Bool { get set }
var hasSeenGamesNewsletterPrompt: Bool { get set }
var hasSeenSaveProjectAlert: Bool { get set }
Expand Down Expand Up @@ -68,15 +66,6 @@ extension KeyValueStoreType {
}
}

public var hasSeenCategoryPersonalizationFlow: Bool {
get {
return self.bool(forKey: AppKeys.hasSeenCategoryPersonalizationFlow.rawValue)
}
set {
self.set(newValue, forKey: AppKeys.hasSeenCategoryPersonalizationFlow.rawValue)
}
}

public var hasDismissedPersonalizationCard: Bool {
get {
return self.bool(forKey: AppKeys.hasDismissedPersonalizationCard.rawValue)
Expand Down
4 changes: 0 additions & 4 deletions Library/ViewModels/LandingViewModel.swift
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,6 @@ public protocol LandingViewModelType {

public final class LandingViewModel: LandingViewModelType, LandingViewModelInputs, LandingViewModelOutputs {
msadoon marked this conversation as resolved.
Show resolved Hide resolved
public init() {
self.viewDidLoadProperty.signal.observeValues {
AppEnvironment.current.userDefaults.hasSeenCategoryPersonalizationFlow = true
}

self.goToCategorySelection = self.getStartedButtonTappedProperty.signal
}

Expand Down
12 changes: 0 additions & 12 deletions Library/ViewModels/LandingViewModelTests.swift
Original file line number Diff line number Diff line change
Expand Up @@ -26,16 +26,4 @@ final class LandingViewModelTests: TestCase {

self.goToCategorySelection.assertValueCount(1)
msadoon marked this conversation as resolved.
Show resolved Hide resolved
}

func testHasSeenCategoryPersonalizationFlowPropertyIsSet() {
let mockKVStore = MockKeyValueStore()

withEnvironment(userDefaults: mockKVStore) {
self.vm.inputs.viewDidLoad()

self.scheduler.run()

XCTAssertTrue(mockKVStore.hasSeenCategoryPersonalizationFlow)
}
}
}