-
Notifications
You must be signed in to change notification settings - Fork 248
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(fcm): enable direct_boot_ok as bool in Android config #638
feat(fcm): enable direct_boot_ok as bool in Android config #638
Conversation
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
347d894
to
3c85b5c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! Looks pretty good!
Let's remove the changes in .idea
from this PR
.idea/.gitignore
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's remove the .idea
directory from this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
.idea/firebase-admin-go.iml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's remove the .idea
directory and all the files under it from this PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! LGTM!
Let me get the internal API approval process started before we can merge this change
Discussion
Testing
go test -test.short firebase.google.com/go/...
? firebase.google.com/go/v4/errorutils [no test files]
? firebase.google.com/go/v4/integration/internal [no test files]
? firebase.google.com/go/v4/snippets [no test files]
ok firebase.google.com/go/v4 3.946s
ok firebase.google.com/go/v4/appcheck 1.608s
ok firebase.google.com/go/v4/auth 4.664s
ok firebase.google.com/go/v4/auth/hash 3.035s
ok firebase.google.com/go/v4/db 2.847s
ok firebase.google.com/go/v4/iid 2.115s
ok firebase.google.com/go/v4/integration/auth 4.185s
ok firebase.google.com/go/v4/integration/db 4.624s
ok firebase.google.com/go/v4/integration/firestore 5.664s
ok firebase.google.com/go/v4/integration/iid 5.171s
ok firebase.google.com/go/v4/integration/messaging 6.141s
ok firebase.google.com/go/v4/integration/storage 6.411s
ok firebase.google.com/go/v4/internal 7.386s
ok firebase.google.com/go/v4/messaging 6.563s
ok firebase.google.com/go/v4/storage 7.937s
API Changes