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

Conversation

@nheagy
Copy link
Contributor

@nheagy nheagy commented Jun 26, 2018

Fixes wordpress-mobile/WordPress-iOS#8388

simulator screen shot - iphone x - 2018-06-26 at 12 12 09

When logging in with restrictToWPCom turned on, and the uses selects to login via Google, we now hide the option to login via address when there is an error. This will improve error recovery for these users.

To test:

  • Login to the app using a self-hosted site
  • On the Me tab, tap to login to wpcom
  • Chose to login via Google
  • Force an error (I modify googleLoginTapped() in LoginEmailViewController)
  • Ensure that the option to retry via site address is gone, and that the other two options still work
  • Ensure that the three options all appear and work when login is not restricted to wpcom

@nheagy nheagy added the bug Something isn't working label Jun 26, 2018
@nheagy nheagy self-assigned this Jun 26, 2018
@nheagy nheagy requested a review from jleandroperez June 26, 2018 18:19
Copy link
Contributor

@jleandroperez jleandroperez left a comment

Choose a reason for hiding this comment

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

Verified in WPiOS 10.4 (develop!).

:shipit:

@nheagy
Copy link
Contributor Author

nheagy commented Jun 28, 2018

Thanks @jleandroperez!

@nheagy nheagy merged commit b039eb8 into master Jun 28, 2018
@nheagy nheagy deleted the fix/wpcom-restriction-in-social-error branch June 28, 2018 16:06
crazytonyli added a commit that referenced this pull request Nov 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants