Skip to content

Conversation

grantland
Copy link
Contributor

Publicize + JavaDoc. Required for open sourcing ParseFacebookUtils and ParseTwitterUtils.

Copy link
Contributor

Choose a reason for hiding this comment

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

This should be deauthenticateInBackground?

Same with restoreAuthenticationAsync

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good catch!

@grantland
Copy link
Contributor Author

@hallucinogen updated

@hallucinogen
Copy link
Contributor

Looking good! @nlutsenko should double-check tho since this is a public API.

@nlutsenko
Copy link
Contributor

LGTM

@nlutsenko nlutsenko assigned grantland and unassigned nlutsenko Sep 11, 2015
@grantland grantland force-pushed the grantland.authentication_public branch from 3b95f36 to bb0f357 Compare September 11, 2015 19:02
Publicize + JavaDoc. Required for open sourcing ParseFacebookUtils and ParseTwitterUtils
@grantland grantland force-pushed the grantland.authentication_public branch from bb0f357 to c79d8c1 Compare September 11, 2015 19:03
grantland added a commit that referenced this pull request Sep 11, 2015
…ublic

Publicize Third Party Authentication methods
@grantland grantland merged commit 4684053 into master Sep 11, 2015
@grantland grantland deleted the grantland.authentication_public branch September 11, 2015 19:08
@facebook-github-bot
Copy link

@grantland updated the pull request.

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.

4 participants