Skip to content
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

WordPressAuthenticator: Wiring the latest Delegate's API #9750

Merged
merged 4 commits into from
Jul 12, 2018

Conversation

jleandroperez
Copy link
Contributor

Details:

In this PR we're dropping a callback parameter which is not currently used by the WordPressAuthenticator. No new feature is being introduced.

To test:

  1. Verify the branch builds correctly
  2. Perform a fresh install, and verify the Login flow works as expected.

NOTE: Once this PR is approved, we'll swap to a formal WordPressAuthenticator release in the podfile.

cc @bummytime @mindgraffiti
(Thanks in advance!!!)

Copy link
Contributor

@mindgraffiti mindgraffiti left a comment

Choose a reason for hiding this comment

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

Login flow ☑️
Code changes ☑️

:shipit: !

@jleandroperez jleandroperez merged commit b5f4a7f into develop Jul 12, 2018
@jleandroperez jleandroperez deleted the issue/updating-authenticator-api branch July 12, 2018 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants