-
Notifications
You must be signed in to change notification settings - Fork 52
DOCSP-29676 v5.4 update #675
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-29676 v5.4 update #675
Conversation
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.
good job. tiny things
source/whats-new.txt
Outdated
|
||
.. important:: Deprecation Notice | ||
|
||
- The ``collStats`` helper is deprecated. Use the :manual:`$collStats |
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: Looking at the JIRA ticket, collStats
is a helper command. Might be useful to specify here.
- The ``collStats`` helper is deprecated. Use the :manual:`$collStats | |
- The ``collStats`` helper command is deprecated. Use the :manual:`$collStats |
source/whats-new.txt
Outdated
- Methods of ``db.command()`` that were incorrectly included in the | ||
typescript interface have been deprecated. |
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.
I: I think these were options, not methods.
- Methods of ``db.command()`` that were incorrectly included in the | |
typescript interface have been deprecated. | |
- The TypeScript interface passed to the ``db.command()`` method incorrectly | |
includes certain options. These options have been deprecated. |
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.
Oops, good catch. Changed
source/whats-new.txt
Outdated
- Methods of ``db.command()`` that were incorrectly included in the | ||
typescript interface have been deprecated. | ||
|
||
- The ``ChangeStream.tryNext`` method now uses the ``TChange`` schema |
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 ``ChangeStream.tryNext`` method now uses the ``TChange`` schema | |
- The ``ChangeStream.tryNext()`` method now uses the ``TChange`` schema |
source/whats-new.txt
Outdated
- The ``ChangeStream.tryNext`` method now uses the ``TChange`` schema | ||
generic instead of the untyped ``Document`` interface. |
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: Something like:
- The ``ChangeStream.tryNext`` method now uses the ``TChange`` schema | |
generic instead of the untyped ``Document`` interface. | |
- The ``ChangeStream.tryNext`` method now uses the schema-specific | |
``TChange`` generic type instead of the ``Document`` interface. |
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-29676
Staging - https://docs-mongodbcom-staging.corp.mongodb.com/node/docsworker-xlarge/DOCSP-29676-v5.4-release/
Self-Review Checklist