Skip to content

Login: retain email address after failed Google login attempts - #8021

Merged
nheagy merged 2 commits into
developfrom
feature/7675-retain-email-after-error
Oct 23, 2017
Merged

Login: retain email address after failed Google login attempts#8021
nheagy merged 2 commits into
developfrom
feature/7675-retain-email-after-error

Conversation

@nheagy

@nheagy nheagy commented Oct 20, 2017

Copy link
Copy Markdown
Contributor

Refs #7675

Small change so that we keep the Google email address of failed login attempts to prefill on subsequent login or signup screens.

To test:

  • Use the Google button to login to a Google account with no associated WordPress.com account.
  • On the error screen, pick an option and ensure that the Google email address is correctly prefilled
  • Repeat for the other two options

Needs review: @aerych

@nheagy nheagy added the Login label Oct 20, 2017
@nheagy nheagy added this to the 8.7 milestone Oct 20, 2017
@nheagy nheagy self-assigned this Oct 20, 2017
@nheagy
nheagy requested a review from aerych October 20, 2017 15:49
@nheagy
nheagy force-pushed the feature/7675-retain-email-after-error branch from 2bfbfaf to 3828004 Compare October 20, 2017 15:49
@nheagy nheagy mentioned this pull request Oct 20, 2017
17 tasks

extension LoginEmailViewController: LoginSocialErrorViewControllerDelegate {
private func cleanupAfterSocialErrors() {
loginFields.username = ""

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we'd still want to do this from retryWithEmail... otherwise it might seem a bit weird to tap the "Try with another email" option and see that address in the text field.

@nheagy

nheagy commented Oct 20, 2017

Copy link
Copy Markdown
Contributor Author

@aerych I agree! Also, turns out Signup wasn't working, still. Fixed that too.

Ready for a second look.

@aerych

aerych commented Oct 20, 2017

Copy link
Copy Markdown
Contributor

Nice.
I tested again and noticed that the email address was still being populated in the textfield even though its being unset in loginFields. It looks like configureEmailField is called by viewWillAppear when the Google controller is dismissed. So the email address is already present before the error screen is even shown. Should we add another call to configureEmailFieldin retryWithEmail so its removed?

@nheagy

nheagy commented Oct 22, 2017

Copy link
Copy Markdown
Contributor Author

@aerych I'm only seeing the email get filled when tapping "Try with the site address" and "Sign up", which seems unavoidable. When tapping "Try with another email" it's being cleared for me. Can you double check what you're seeing?

Fwiw, the value is set to "" before dismiss() is called, which should then also be before viewWillAppear: is called.

@aerych

aerych commented Oct 22, 2017

Copy link
Copy Markdown
Contributor

Can you double check

Huh. I'm not seeing it now. :(

🤷‍♂️

:shipit: then :) If its a heisenbug we'll worry about it if it reappears.

@nheagy

nheagy commented Oct 23, 2017

Copy link
Copy Markdown
Contributor Author

Thanks, @aerych!

@nheagy
nheagy merged commit bb08e39 into develop Oct 23, 2017
@nheagy
nheagy deleted the feature/7675-retain-email-after-error branch October 23, 2017 00:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants