Skip to content

Conversation

@hiranya911
Copy link
Contributor

This is a breaking change to remove the deprecated FirebaseCredential and FirebaseCredentials public APIs. Developers should use GoogleCredentials for initializing the SDK.

go/firebase-java-googlecreds

Copy link
Contributor

@avishalom avishalom left a comment

Choose a reason for hiding this comment

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

LGTM !

@avishalom avishalom assigned hiranya911 and unassigned avishalom Mar 9, 2018
@hiranya911 hiranya911 merged commit f8f33b4 into v6 Mar 9, 2018
@hiranya911 hiranya911 deleted the hkj-remove-creds branch March 9, 2018 17:34
hiranya911 added a commit that referenced this pull request May 7, 2018
* Adding the sync APIs for FirebaseAuth

* Added more tests; Added sync APIs for FirebaseMessaging

* Removing Task references from database, iid and fcm APIs

* Fixing a typo

* Minor code clean up

* Updated javadocs; Renamed internal helpers of FirebaseMessaging for consistency

* Removed the deprecated FirebaseCredential API (#149)

* Removing the Task API (#152)

* Removed the deprecated FirebaseCredential API

* Removing the deprecated Task API

* Dropping Support for App Engine Java 7 Runtime (#153)

* Dropping support for GAE 7

* Removed GaeThreadFactory, GaeExecutorService and RevivingScheduledExecutor

* Removed the deprecated FirebaseCredential API

* Removing GAE java7 related APIs (GaeThreadFactory, RevivingScheduledExecutor)

* Removed GaePlatform implementation

* Added FirebaseScheduledExecutor

* Updated documentation

* Some minor nits from code reviews

* Calling super method in DefaultRunLoop executor

* Removing Deprecated LogWrapper API (#154)

* Dropping support for GAE 7

* Removed GaeThreadFactory, GaeExecutorService and RevivingScheduledExecutor

* Removed the deprecated FirebaseCredential API

* Removing GAE java7 related APIs (GaeThreadFactory, RevivingScheduledExecutor)

* Removed GaePlatform implementation

* Added FirebaseScheduledExecutor

* Updated documentation

* Removing LogWrapper API

* Removing PrefixedLogger

* Removing test config file

* Updated CHANGELOG

* Minor clean ups pointed out in the code review

* Minor API doc fixes (#171)
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.

2 participants