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

Tweaks on the whole register/auth flow #25

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

iamntz
Copy link

@iamntz iamntz commented Mar 3, 2015

Some kind of continuation of #22 and #23 (this PR also includes the change in those PRs)

Some notes:

  • for some reasons, register.php didn't displayed any Diffs, so i deleted the file and recreated as WP_OauthRegister.php (that's why the file looks like 100% changed);
  • refactored linked accounts section into its own class, in order to be easily extensible;
  • Added various isset statements to prevent notices & warnings;
  • added various hooks to allow integrating other plugins (e.g. Register Redux)
  • it's tested only with facebook. Any other testers would be more than welcomed ;)

That's about all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant