Skip to content

Commit b5234ea

Browse files
Merge pull request #7999 from woocommerce/issue/7997-simplifiedLogin-prologue
Simplified login: Prologue screen updates
2 parents 30bc4e9 + 9d9b50d commit b5234ea

File tree

7 files changed

+15
-26
lines changed

7 files changed

+15
-26
lines changed

Experiments/Experiments/DefaultFeatureFlagService.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ public struct DefaultFeatureFlagService: FeatureFlagService {
1818
case .shippingLabelsOnboardingM1:
1919
return buildConfig == .localDeveloper || buildConfig == .alpha
2020
case .newToWooCommerceLinkInLoginPrologue:
21-
return true
21+
return false
2222
case .loginPrologueOnboarding:
2323
return true
2424
case .loginErrorNotifications:

Podfile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@ target 'WooCommerce' do
6565
pod 'Gridicons', '~> 1.2.0'
6666

6767
# To allow pod to pick up beta versions use -beta. E.g., 1.1.7-beta.1
68-
pod 'WordPressAuthenticator', '~> 4.0.0'
69-
# pod 'WordPressAuthenticator', :git => 'https://github.com/wordpress-mobile/WordPressAuthenticator-iOS.git', :commit => ''
68+
pod 'WordPressAuthenticator', '~> 4.1.0-beta.1'
69+
# pod 'WordPressAuthenticator', :git => 'https://github.com/wordpress-mobile/WordPressAuthenticator-iOS.git', :commit => 'f24fd9c6dbc73cfee583a7eba16fa81e2948fa43'
7070
# pod 'WordPressAuthenticator', :git => 'https://github.com/wordpress-mobile/WordPressAuthenticator-iOS.git', :branch => ''
7171
# pod 'WordPressAuthenticator', :path => '../WordPressAuthenticator-iOS'
7272

Podfile.lock

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ PODS:
4242
- WordPress-Aztec-iOS (1.11.0)
4343
- WordPress-Editor-iOS (1.11.0):
4444
- WordPress-Aztec-iOS (= 1.11.0)
45-
- WordPressAuthenticator (4.0.0):
45+
- WordPressAuthenticator (4.1.0-beta.1):
4646
- CocoaLumberjack (~> 3.5)
4747
- GoogleSignIn (~> 6.0.1)
4848
- Gridicons (~> 1.0)
@@ -91,7 +91,7 @@ DEPENDENCIES:
9191
- Sourcery (~> 1.0.3)
9292
- StripeTerminal (~> 2.7)
9393
- WordPress-Editor-iOS (~> 1.11.0)
94-
- WordPressAuthenticator (~> 4.0.0)
94+
- WordPressAuthenticator (~> 4.1.0-beta.1)
9595
- WordPressKit (~> 4.49.0)
9696
- WordPressShared (~> 1.15)
9797
- WordPressUI (~> 1.12.5)
@@ -101,6 +101,8 @@ DEPENDENCIES:
101101
- ZendeskSupportSDK (~> 5.0)
102102

103103
SPEC REPOS:
104+
https://github.com/wordpress-mobile/cocoapods-specs.git:
105+
- WordPressAuthenticator
104106
trunk:
105107
- Alamofire
106108
- AppAuth
@@ -123,7 +125,6 @@ SPEC REPOS:
123125
- UIDeviceIdentifier
124126
- WordPress-Aztec-iOS
125127
- WordPress-Editor-iOS
126-
- WordPressAuthenticator
127128
- WordPressKit
128129
- WordPressShared
129130
- WordPressUI
@@ -161,7 +162,7 @@ SPEC CHECKSUMS:
161162
UIDeviceIdentifier: af4e11e25a2ea670078e2bd677bb0e8144f9f063
162163
WordPress-Aztec-iOS: 050b34d4c3adfb7c60363849049b13d60683b348
163164
WordPress-Editor-iOS: 304098424f1051cb271546c99f906aac296b1b81
164-
WordPressAuthenticator: c0bdeec612e4e80d76ac7849f89d8405c1d43654
165+
WordPressAuthenticator: 1f3dabd0f7e95afca4efdd3b528417419b66250b
165166
WordPressKit: 96deb6ba37ea5eaec4ddcaa53eca04d653246152
166167
WordPressShared: 5477f179c7fe03b5d574f91adda66f67d131827e
167168
WordPressUI: c5be816f6c7b3392224ac21de9e521e89fa108ac
@@ -177,6 +178,6 @@ SPEC CHECKSUMS:
177178
ZendeskSupportProvidersSDK: 2bdf8544f7cd0fd4c002546f5704b813845beb2a
178179
ZendeskSupportSDK: 3a8e508ab1d9dd22dc038df6c694466414e037ba
179180

180-
PODFILE CHECKSUM: 56e66ae89d30a0b6cfc246d43c71ba38eff01e0d
181+
PODFILE CHECKSUM: 4286e3bac570110f96d064f4359aec6a46b2406a
181182

182183
COCOAPODS: 1.11.3

WooCommerce/Classes/Authentication/AuthenticationConstants.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -121,10 +121,10 @@ struct AuthenticationConstants {
121121
comment: "Legal disclaimer for signing up. The underscores _..._ denote underline."
122122
)
123123

124-
/// Create a Store button in Simplified login flow
124+
/// Get Started button in Simplified login flow
125125
///
126126
static let createSiteButtonTitle = NSLocalizedString(
127-
"Create a Store",
127+
"Get Started",
128128
comment: "Button displayed on the prologue screen of the simplified login flow to create a new store"
129129
)
130130

WooCommerce/Classes/Authentication/AuthenticationManager.swift

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,6 @@ class AuthenticationManager: Authentication {
5555
func initialize(loggedOutAppSettings: LoggedOutAppSettingsProtocol) {
5656
let isWPComMagicLinkPreferredToPassword = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.loginMagicLinkEmphasis)
5757
let isWPComMagicLinkShownAsSecondaryActionOnPasswordScreen = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.loginMagicLinkEmphasisM2)
58-
let isFeatureCarouselShown = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.loginPrologueOnboarding) == false
59-
|| (loggedOutAppSettings.hasFinishedOnboarding == true &&
60-
ServiceLocator.featureFlagService.isFeatureFlagEnabled(.simplifiedLoginFlowI1) == false)
6158
let isSimplifiedLoginI1Enabled = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.simplifiedLoginFlowI1)
6259
let isStoreCreationMVPEnabled = ServiceLocator.featureFlagService.isFeatureFlagEnabled(.storeCreationMVP)
6360
let configuration = WordPressAuthenticatorConfiguration(wpcomClientId: ApiCredentials.dotcomAppId,
@@ -117,7 +114,7 @@ class AuthenticationManager: Authentication {
117114
navBarBadgeColor: .primary,
118115
navBarBackgroundColor: .appBar,
119116
prologueTopContainerChildViewController:
120-
LoginPrologueViewController(isFeatureCarouselShown: isFeatureCarouselShown),
117+
LoginPrologueViewController(isFeatureCarouselShown: false),
121118
statusBarStyle: .default)
122119

123120
let getStartedInstructions = isSimplifiedLoginI1Enabled ?

WooCommerce/Classes/Authentication/Prologue/LoginProloguePages.swift

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,7 @@ enum LoginProloguePageType: CaseIterable {
2626
case .reviews:
2727
return NSLocalizedString("Monitor and approve your product reviews",
2828
comment: "Caption displayed in promotional screens shown during the login flow.")
29-
case .getStarted:
30-
return NSLocalizedString("Let’s get started!",
31-
comment: "Caption displayed in the prologue screen shown after onboarding during the login flow.")
32-
case .simplifiedLoginI1Intro:
29+
case .getStarted, .simplifiedLoginI1Intro:
3330
return NSLocalizedString("WooCommerce is a customizable, open-source eCommerce platform built on WordPress.",
3431
comment: "Caption displayed in the simplified prologue screen")
3532
}
@@ -145,7 +142,7 @@ private extension LoginProloguePageTypeViewController {
145142

146143
// Label style & layout
147144
titleLabel.font = {
148-
if pageType == .simplifiedLoginI1Intro {
145+
if pageType == .simplifiedLoginI1Intro || pageType == .getStarted {
149146
return .headline
150147
} else if showsSubtitle {
151148
return .font(forStyle: .title2, weight: .semibold)

WooCommerce/UITestsFoundation/Screens/Login/PrologueScreen.swift

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -30,13 +30,7 @@ public final class PrologueScreen: ScreenObject {
3030

3131
extension PrologueScreen {
3232
static func findContinueButton(in app: XCUIApplication) -> XCUIElement {
33-
let continueButton = app.buttons["Prologue Continue Button"]
34-
if continueButton.waitForExistence(timeout: 1) {
35-
return continueButton
36-
} else {
37-
// On simplified login flow, the button has different identifier
38-
return app.buttons["Prologue Log In Button"]
39-
}
33+
app.buttons["Prologue Continue Button"]
4034
}
4135

4236
public static func isSiteAddressLoginAvailable(in app: XCUIApplication = .init()) -> Bool {

0 commit comments

Comments
 (0)