fix(NODE-3344): allow setting defaultTransactionOptions
with POJO rather than ReadConcern instance
#3032
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Re: Automattic/mongoose#10805
Description
Right now,
startSession({ defaultTransactionOptions: { readConcern: { level: 'snapshot' } } })
fails to compile with aProperty 'toJSON' is missing in type '{ level: string; }' but required in type 'ReadConcern'.
error. This seems like something that should change.What is changing?
TransactionOptions.readConcern
is now aReadConcernLike
, which allows using strings or{ level }
object.Is there new documentation needed for these changes?
What is the motivation for this change?
Make
TransactionOptions
more user-friendly and line up withCursor#withReadConcern()
andCollectionOptions
.Double check the following
npm run check:lint
script<type>(NODE-xxxx)<!>: <description>