Skip to content

Commit ff16474

Browse files
authored
Fix Jetpack migration overlay being shown in the Reader app (#24489)
1 parent a92183c commit ff16474

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

Tests/KeystoneTests/Tests/Jetpack/JetpackFeaturesRemovalCoordinatorTests.swift

+2-2
Original file line numberDiff line numberDiff line change
@@ -357,7 +357,7 @@ final class JetpackFeaturesRemovalCoordinatorTests: CoreDataTestCase {
357357

358358
// When
359359
// assume that we're requesting from the WordPress app.
360-
let phase = JetpackFeaturesRemovalCoordinator.generalPhase(featureFlagStore: store, isJetpack: false)
360+
let phase = JetpackFeaturesRemovalCoordinator.generalPhase(featureFlagStore: store, app: .wordpress)
361361

362362
// Then
363363
XCTAssertEqual(phase, .staticScreens)
@@ -372,7 +372,7 @@ final class JetpackFeaturesRemovalCoordinatorTests: CoreDataTestCase {
372372

373373
// When
374374
// assume that we're requesting from the Jetpack app.
375-
let phase = JetpackFeaturesRemovalCoordinator.generalPhase(featureFlagStore: store, isJetpack: true)
375+
let phase = JetpackFeaturesRemovalCoordinator.generalPhase(featureFlagStore: store, app: .jetpack)
376376

377377
// Then
378378
XCTAssertEqual(phase, .normal)

WordPress/Classes/ViewRelated/Jetpack/Branding/Coordinator/JetpackFeaturesRemovalCoordinator.swift

+6-5
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import Foundation
2+
import BuildSettingsKit
23
import WordPressShared
34

45
/// A class containing convenience methods for the the Jetpack features removal experience
@@ -92,9 +93,9 @@ public class JetpackFeaturesRemovalCoordinator: NSObject {
9293
static var currentAppUIType: RootViewCoordinator.AppUIType?
9394

9495
static func generalPhase(featureFlagStore: RemoteFeatureFlagStore = RemoteFeatureFlagStore(),
95-
isJetpack: Bool = AppConfiguration.isJetpack) -> GeneralPhase {
96-
if isJetpack {
97-
return .normal // Always return normal for Jetpack
96+
app: AppBrand = BuildSettings.current.brand) -> GeneralPhase {
97+
guard app == .wordpress else {
98+
return .normal // Show migration only for the WordPress app
9899
}
99100

100101
if AccountHelper.noWordPressDotComAccount {
@@ -124,8 +125,8 @@ public class JetpackFeaturesRemovalCoordinator: NSObject {
124125
}
125126

126127
static func siteCreationPhase(featureFlagStore: RemoteFeatureFlagStore = RemoteFeatureFlagStore()) -> SiteCreationPhase {
127-
if AppConfiguration.isJetpack {
128-
return .normal // Always return normal for Jetpack
128+
guard BuildSettings.current.brand == .wordpress else {
129+
return .normal // Show migration only for the WordPress app
129130
}
130131

131132
if RemoteFeatureFlag.jetpackFeaturesRemovalPhaseNewUsers.enabled(using: featureFlagStore)

0 commit comments

Comments
 (0)