Closed nheagy closed 7 years ago
Did a bit of testing this morning with the feature. Most everything works as expected. :) Just a few nitpicks:
[ ] First time google button is tapped it takes a moment before the new UIAlertView appears. Maybe call configureViewLoading(true)
right away to start the spinner and disable things to prevent extra taps.
[ ] 2fa if I enter invalid numeric input I see the desired error but if I enter letters a 400 response comes back and we show the error view with a cryptic message. Maybe sanitize input so it only accepts numerics? Otoh, the keyboard is numeric so maybe that's enough? Thoughts?
[ ] if I tap back to the email screen from the password screen or the 2fa screen the next button is still showing a spinner and the form is disabled. Probably should call configureViewLoading(false)
from needsMultifactorCode and existingUserNeedsConnection.
Should also disconnect google but not exactly sure where is best to do so.
[ ] Trying to sign in with a google account not matching a wpcom account shows the error modal but the error message is the "This operation couldn't be completed. (WordPressComOAuthError error 0.)" message. Tapping the google button again seems to try the same login attempt (same google account). Need to disconnect google.
Maybe sanitize input so it only accepts numerics? Otoh, the keyboard is numeric so maybe that's enough?
You're right, we should limit to numbers. iirc on iPad numeric keyboard will still allow letters, and so will pasting. 👍
I agree with the others. Thanks for the testing @aerych!
Add Google account login to the existing login flow.
Development tasks
Features
Other issues
Done
(#8012)Mockup