-
Notifications
You must be signed in to change notification settings - Fork 48
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
DOCSP-43533 - Version 6.9 #902
Conversation
✅ Deploy Preview for docs-node ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
source/whats-new.txt
Outdated
.. code-block:: javascript | ||
:copyable: true | ||
|
||
const client = new MongoClient(process.env.MONGODB_URI, { autoSelectFamily: true, autoSelectFamilyTimeout: 100 }); |
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: not sure we use an env variable in most other places. In this instance, I think you can omit the connection string parameter altogether and the driver uses localhost 27017. Also, newline where possible to reduce the scroll
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.
The API docs says URL is required? I can just use a variable.
source/whats-new.txt
Outdated
This optimization doesn't apply if you provide a transform to the ``cursor.map()`` method | ||
before calling the ``toArray()`` method. | ||
- Fixes mixed use of the ``cursor.next()`` method along with ``for await`` syntax. | ||
- Fixes the ``enableUtf8Validation`` when deserializing BSON. |
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: more info needed. is there an error?
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.
added more
Co-authored-by: Rea Rustagi <[email protected]>
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.
pretty much ready, but wondering if there are content changes needed for some of the items!
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
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-43533
Staging:
Self-Review Checklist