Skip to content

Conversation

@jasonhu-g
Copy link
Contributor

Move FIRInstanceIDInstanceProvider into the private header file for components to look up an instance of FIRInstanceID. This is needed when a components loads with componentsToRegister and depends on instanceID when initializing.

…omponents to look up an instance of FIRInstanceID
*
* @return YES if valid checkin preferences exist in memory else NO.
*/
- (BOOL)hasValidCheckinInfo;
Copy link
Member

Choose a reason for hiding this comment

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

Was this intentional?

Copy link
Contributor Author

@jasonhu-g jasonhu-g Oct 10, 2019

Choose a reason for hiding this comment

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

Yep, so Travis CI failed because XCode generated a warning where FIRInstanceID does not have an implementation for hasValidCheckinInfo. I think this is old code since there's no references to that method in FIRInstanceID anywhere. It looks like the code got refactored and that method no lives in FIRInstanceIDCheckinPreferences.

@jasonhu-g
Copy link
Contributor Author

This change is no longer needed with a fix Core initialization (#4030)

@jasonhu-g jasonhu-g closed this Oct 14, 2019
@firebase firebase locked and limited conversation to collaborators Nov 14, 2019
@jasonhu-g jasonhu-g deleted the moveIIDProviderIntoPrivateHeader branch January 8, 2020 16:54
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