-
Notifications
You must be signed in to change notification settings - Fork 150
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
chore: release 4.0.0 #1234
chore: release 4.0.0 #1234
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1234 +/- ##
=======================================
Coverage 98.50% 98.50%
=======================================
Files 30 30
Lines 18545 18545
Branches 1422 1422
=======================================
Hits 18268 18268
Misses 274 274
Partials 3 3 Continue to review full report at Codecov.
|
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.
lgtm
CHANGELOG.md
Outdated
### Features | ||
|
||
* add support for serialization to BigInt `(via settings({useBigInt: true})` (#1016) | ||
* add support for set() with SetOptions when using `FirestoreDataConverte` (#1087) |
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.
s/Converte/Converter
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
CHANGELOG.md
Outdated
### ⚠ BREAKING CHANGES | ||
|
||
* drop Node 8 support (#1006) | ||
* the `FirestoreDataConverter` is now called with a `QueryDocumentSnapshot` instead of `DocumentData` (#965) |
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.
add .fromFirestore()
to FirestoreDataConverter
?
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
Are there plans to add this release to https://github.com/firebase/firebase-admin-node? |
For compatibility with other Firebase offerings that continue to rely on Node 8 support, the upgrade to |
🤖 I have created a release *beep* *boop*
4.0.0 (2020-06-24)
⚠ BREAKING CHANGES
FirestoreDataConverter.fromFirestore()
is now called with aQueryDocumentSnapshot
instead ofDocumentData
(feat!: use QueryDocumentSnapshot in FirestoreDataConverter #965)Features
(via settings({useBigInt: true})
( feat: support BigInt #1016)FirestoreDataConverter
(feat: add support for set() with SetOptions when using FirestoreDataConverter #1087)Bug Fix
@grpc/grpc-js
are installed (Firestore SDK does not play well with different versions of grpc-js installed #1233)This PR was generated with Release Please.