Skip to content
This repository was archived by the owner on Feb 5, 2025. It is now read-only.

Commit ce4e10a

Browse files
authored
Merge pull request #742 from wordpress-mobile/wcios/wplogin-without-site-credential
Add configuration to disable site credential login on Get Started screen
2 parents 49b7e4c + 27b3eb0 commit ce4e10a

File tree

5 files changed

+18
-9
lines changed

5 files changed

+18
-9
lines changed

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ _None._
3838

3939
### New Features
4040

41-
_None._
41+
- New configuration to disable site credential login on Get Started screen for the site address login flow [#742]
4242

4343
### Bug Fixes
4444

Podfile.lock

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ PODS:
2222
- SVProgressHUD (2.2.5)
2323
- SwiftLint (0.49.1)
2424
- UIDeviceIdentifier (2.2.0)
25-
- WordPressAuthenticator (5.3.0):
25+
- WordPressAuthenticator (5.4.0-beta.1):
2626
- GoogleSignIn (~> 6.0.1)
2727
- Gridicons (~> 1.0)
2828
- "NSURL+IDN (= 0.4)"
@@ -94,7 +94,7 @@ SPEC CHECKSUMS:
9494
SVProgressHUD: 1428aafac632c1f86f62aa4243ec12008d7a51d6
9595
SwiftLint: 32ee33ded0636d0905ef6911b2b67bbaeeedafa5
9696
UIDeviceIdentifier: f33af270ba9045ea18b31d9aab88e42a0082ea67
97-
WordPressAuthenticator: d33383ff4e813d23d7891168668322abc10be77c
97+
WordPressAuthenticator: cb33aace25e044c96b40957826df5e9060cb0924
9898
WordPressKit: 08da0bc981f6398ef7a32e523fd054de7d7c7069
9999
WordPressShared: 04403b43f821c4ed2b84a2112ef9f64f1e7cdceb
100100
WordPressUI: 1cf47a3b78154faf69caa18569ee7ece1e510fa0

WordPressAuthenticator.podspec

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
Pod::Spec.new do |s|
44
s.name = 'WordPressAuthenticator'
5-
s.version = '5.3.0'
5+
s.version = '5.4.0-beta.1'
66

77
s.summary = 'WordPressAuthenticator implements an easy and elegant way to authenticate your WordPress Apps.'
88
s.description = <<-DESC

WordPressAuthenticator/Authenticator/WordPressAuthenticatorConfiguration.swift

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,11 @@ public struct WordPressAuthenticatorConfiguration {
168168
///
169169
let enableSiteAddressLoginOnlyInPrologue: Bool
170170

171+
/// If enabled, the email login screen should not hide the site credential login CTA.
172+
/// This is `true` by default.
173+
///
174+
let enableSiteCredentialLoginForJetpackSites: Bool
175+
171176
/// Designated Initializer
172177
///
173178
public init (wpcomClientId: String,
@@ -203,6 +208,7 @@ public struct WordPressAuthenticatorConfiguration {
203208
enableManualSiteCredentialLogin: Bool = false,
204209
useEnterEmailAddressAsStepValueForGetStartedVC: Bool = false,
205210
enableSiteAddressLoginOnlyInPrologue: Bool = false,
211+
enableSiteCredentialLoginForJetpackSites: Bool = true,
206212
googleLoginWithoutSDK: Bool = false
207213
) {
208214

@@ -240,5 +246,6 @@ public struct WordPressAuthenticatorConfiguration {
240246
self.enableManualSiteCredentialLogin = enableManualSiteCredentialLogin
241247
self.useEnterEmailAddressAsStepValueForGetStartedVC = useEnterEmailAddressAsStepValueForGetStartedVC
242248
self.enableSiteAddressLoginOnlyInPrologue = enableSiteAddressLoginOnlyInPrologue
249+
self.enableSiteCredentialLoginForJetpackSites = enableSiteCredentialLoginForJetpackSites
243250
}
244251
}

WordPressAuthenticator/Unified Auth/View Related/Get Started/GetStartedViewController.swift

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -810,11 +810,13 @@ private extension GetStartedViewController {
810810
accessibilityIdentifier: ButtonConfiguration.Continue.accessibilityIdentifier,
811811
onTap: handleSubmitButtonTapped)
812812

813-
// Setup Sign in with site credentials button
814-
buttonViewController.setupBottomButton(attributedTitle: WPStyleGuide.formattedSignInWithSiteCredentialsString(),
815-
isPrimary: false,
816-
accessibilityIdentifier: ButtonConfiguration.SignInWithSiteCredentials.accessibilityIdentifier,
817-
onTap: handleSiteCredentialsButtonTapped)
813+
if configuration.enableSiteCredentialLoginForJetpackSites {
814+
// Setup Sign in with site credentials button
815+
buttonViewController.setupBottomButton(attributedTitle: WPStyleGuide.formattedSignInWithSiteCredentialsString(),
816+
isPrimary: false,
817+
accessibilityIdentifier: ButtonConfiguration.SignInWithSiteCredentials.accessibilityIdentifier,
818+
onTap: handleSiteCredentialsButtonTapped)
819+
}
818820
}
819821

820822
func configureButtonViewControllerWithoutSocialLogin() {

0 commit comments

Comments
 (0)