-
Notifications
You must be signed in to change notification settings - Fork 11
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
UL&S: Remove .showSigninV2
#232
Conversation
Navigate programmatically
Also delete storyboard placeholder from SignupEmailVC
// LoginFacadeDelegate methods for Google Google Sign In | ||
/// Make Google Sign In conform to the LoginFacade protocol. | ||
/// The delegate calls these methods from LoginFacade.m. | ||
/// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is documentation I forgot to update for the previous segue removal, .show2fa
@@ -522,6 +532,9 @@ extension LoginEmailViewController { | |||
googleExistingUserNeedsConnection(email) | |||
} | |||
|
|||
/// After a successful Google Sign In, this method gets called when the user | |||
/// has enabled 2-factor authentication for their WordPress.com account. | |||
/// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is documentation I forgot to update for the previous segue removal, .show2fa
This prepare for segue is no longer used because login in by email and login by google are done using nav controllers rather than calling perform segue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested with wordpress-mobile/WordPress-iOS#13828.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested with the work from wordpress-mobile/WordPress-iOS#13828 (review)
Thank you @ScoutHarris and @bjtitus ! |
Fixes #229
Ref. #182
This PR removes all references to the segue
.showSigninV2
and programmatically navigates the user to theSignupEmailViewController
. There is 1 area where the segue has been removed:To Test - WPiOS
rake dependencies
and ensure there are no errorsshowSigninV2
WordPress-iOS#13828