Skip to content

Conversation

@protocol86
Copy link
Contributor

Adds deprecation messages for linkWithCredential:completion and reauthenicateWithCredential:completion:

Also corrects the deprecation messages for other deprecated APIs where part of the message was being repeated.

@paulb777 paulb777 added this to the 5.0.0 milestone May 1, 2018
Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

lgtm with one nit cc: @ryanwilson

(nullable FIRAuthDataResultCallback)completion
DEPRECATED_MSG_ATTRIBUTE("signInAnonymouslyAndRetrieveDataWithCompletion: is deprecated."
" Please use signInAnonymouslyWithCompletion: for Objective-C or"
DEPRECATED_MSG_ATTRIBUTE(" Please use signInAnonymouslyWithCompletion: for Objective-C or"
Copy link
Member

Choose a reason for hiding this comment

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

Extra space before Please

@protocol86 protocol86 removed the request for review from XiangtianDai May 1, 2018 01:28
@protocol86 protocol86 merged commit d8e9113 into master May 1, 2018
paulb777 pushed a commit that referenced this pull request May 1, 2018
* Adds and modifies deprecation messages

* Addresses Comments
minafarid pushed a commit to minafarid/firebase-ios-sdk that referenced this pull request Jun 6, 2018
* Adds and modifies deprecation messages

* Addresses Comments
minafarid pushed a commit to minafarid/firebase-ios-sdk that referenced this pull request Jun 6, 2018
* Adds and modifies deprecation messages

* Addresses Comments
@paulb777 paulb777 deleted the Auth_Deprecations branch August 8, 2018 17:09
@firebase firebase locked and limited conversation to collaborators Nov 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants