diff --git a/RELEASE-NOTES.txt b/RELEASE-NOTES.txt index 799df12a3d2b..6d17ca12794f 100644 --- a/RELEASE-NOTES.txt +++ b/RELEASE-NOTES.txt @@ -4,7 +4,6 @@ 21.3 ----- -* [***] Adds a smooth, opt-in transition to the Jetpack app. [#19714] * [*] Fixed a minor UI issue where the segmented control under My SIte was being clipped when "Home" is selected. [#19595] * [*] Fixed an issue where the site wasn't removed and the app wasn't refreshed after disconnecting the site from WordPress.com. [#19634] * [*] [internal] Fixed an issue where Jetpack extensions were conflicting with WordPress extensions. [#19665] diff --git a/WordPress/Classes/System/WordPressAppDelegate.swift b/WordPress/Classes/System/WordPressAppDelegate.swift index 7502ac31560d..23f6532f16f4 100644 --- a/WordPress/Classes/System/WordPressAppDelegate.swift +++ b/WordPress/Classes/System/WordPressAppDelegate.swift @@ -328,10 +328,12 @@ class WordPressAppDelegate: UIResponder, UIApplicationDelegate { setupWordPressExtensions() - // Start proactively exporting WP data in the background if the conditions are fulfilled. - // This needs to be called after `setupWordPressExtensions` because it updates the stored data. - DispatchQueue.global().async { - ContentMigrationCoordinator.shared.startOnceIfNeeded() + if FeatureFlag.contentMigration.enabled { + // Start proactively exporting WP data in the background if the conditions are fulfilled. + // This needs to be called after `setupWordPressExtensions` because it updates the stored data. + DispatchQueue.global().async { + ContentMigrationCoordinator.shared.startOnceIfNeeded() + } } shortcutCreator.createShortcutsIf3DTouchAvailable(AccountHelper.isLoggedIn) diff --git a/WordPress/Classes/Utility/BuildInformation/FeatureFlag.swift b/WordPress/Classes/Utility/BuildInformation/FeatureFlag.swift index 8c6ee6fef1d5..e23494719dff 100644 --- a/WordPress/Classes/Utility/BuildInformation/FeatureFlag.swift +++ b/WordPress/Classes/Utility/BuildInformation/FeatureFlag.swift @@ -113,9 +113,9 @@ enum FeatureFlag: Int, CaseIterable, OverrideableFlag { case .jetpackPowered: return true case .jetpackPoweredBottomSheet: - return true + return false case .contentMigration: - return true + return false case .newJetpackLandingScreen: return true case .newWordPressLandingScreen: diff --git a/WordPress/Jetpack/Classes/System/JetpackWindowManager.swift b/WordPress/Jetpack/Classes/System/JetpackWindowManager.swift index 99eab365dce4..1ab8f3a95d07 100644 --- a/WordPress/Jetpack/Classes/System/JetpackWindowManager.swift +++ b/WordPress/Jetpack/Classes/System/JetpackWindowManager.swift @@ -15,21 +15,26 @@ class JetpackWindowManager: WindowManager { } override func showUI(for blog: Blog?) { - // If the user is logged in and has blogs sync'd to their account - if AccountHelper.isLoggedIn && AccountHelper.hasBlogs { - showAppUI(for: blog) - return + if AccountHelper.isLoggedIn { + if AccountHelper.hasBlogs { + // If the user is logged in and has blogs sync'd to their account + showAppUI(for: blog) + return + } else { + // If the user doesn't have any blogs, but they're still logged in, log them out + // the `logOutDefaultWordPressComAccount` method will trigger the `showSignInUI` automatically + AccountHelper.logOutDefaultWordPressComAccount() + return + } } - guard AccountHelper.isLoggedIn else { - self.migrationTracker.trackContentImportEligibility(eligible: shouldImportMigrationData) - shouldImportMigrationData ? importAndShowMigrationContent(blog) : showSignInUI() + guard FeatureFlag.contentMigration.enabled else { + showSignInUI() return } - // If the user doesn't have any blogs, but they're still logged in, log them out - // the `logOutDefaultWordPressComAccount` method will trigger the `showSignInUI` automatically - AccountHelper.logOutDefaultWordPressComAccount() + self.migrationTracker.trackContentImportEligibility(eligible: shouldImportMigrationData) + shouldImportMigrationData ? importAndShowMigrationContent(blog) : showSignInUI() } func importAndShowMigrationContent(_ blog: Blog? = nil) {