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

Adding WordPress provider #655

Merged
merged 2 commits into from Mar 18, 2018

Conversation

Projects
None yet
2 participants
@krombox
Contributor

krombox commented Sep 5, 2017

No description provided.

Show outdated Hide outdated docs/providers/thirdparty.md

@krombox krombox changed the title from Adding Wordpress provider to Adding WordPress provider Mar 18, 2018

@krombox

This comment has been minimized.

Show comment
Hide comment
@krombox

krombox Mar 18, 2018

Contributor

@ramsey have fixed WordPress spelling issue.
For about commented vardump() didn't find any. Changed indentation to tabs 4.
Let me know if still something wrong. Thanks

Contributor

krombox commented Mar 18, 2018

@ramsey have fixed WordPress spelling issue.
For about commented vardump() didn't find any. Changed indentation to tabs 4.
Let me know if still something wrong. Thanks

@ramsey

This comment has been minimized.

Show comment
Hide comment
@ramsey

ramsey Mar 18, 2018

Contributor

I was referring to this line: https://github.com/krombox/oauth2-wordpress/blob/master/src/Provider/WordPressResourceOwner.php#L36

Thanks for correcting the spelling!

Contributor

ramsey commented Mar 18, 2018

I was referring to this line: https://github.com/krombox/oauth2-wordpress/blob/master/src/Provider/WordPressResourceOwner.php#L36

Thanks for correcting the spelling!

@ramsey ramsey merged commit a68854f into thephpleague:master Mar 18, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment