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

Bearer Token Support - #35

Merged
sendhil merged 5 commits into
masterfrom
issues/34-bearer-token-support
Feb 23, 2015
Merged

Bearer Token Support#35
sendhil merged 5 commits into
masterfrom
issues/34-bearer-token-support

Conversation

@jleandroperez

Copy link
Copy Markdown
Contributor

Details:

  • Updates WPComOAuthController so that the user is authenticated via Bearer Token.
  • Fixed a callbackBlock setter (that definitely needed a copy call)

Testing:

  1. Install WPiOS and Sign Into a dotcom account.
  2. Add a new testing WordPress App Here
  3. Open this url in Safari.app: wordpress-oauth-v2://authorize?client_id=[App ID]&redirect_url=[Callback URL]

As a result, WPiOS should be onscreen, and the user should be asked whether if he wants to authorize or not the app.

Note:
WordPressAppDelegate's setupSingleSignOn method should be patched, so that the authToken is initialized.

Closes #34. This is a blocker for this WPiOS issue.

/cc @sendhil

(Thanks in advance!).

@sendhil

sendhil commented Feb 23, 2015

Copy link
Copy Markdown
Contributor

Very nice!

sendhil added a commit that referenced this pull request Feb 23, 2015
@sendhil
sendhil merged commit 3dc7d64 into master Feb 23, 2015
@sendhil
sendhil deleted the issues/34-bearer-token-support branch February 23, 2015 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants