-
Notifications
You must be signed in to change notification settings - Fork 24
DOCSP-44801: Client-side projection breaking v3.0 breaking change #321
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-44801: Client-side projection breaking v3.0 breaking change #321
Conversation
✅ Deploy Preview for mongodb-docs-csharp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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 with a couple minor suggestions
source/upgrade/v3.txt
Outdated
- Queries that use client-side projections will throw an ``ExpressionNotSupportedException`` | ||
by default with v3.0. To enable client-side projections, set the |
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: add "error"; also since this is in the v3.0 section, I don't think you need to specify that again
- Queries that use client-side projections will throw an ``ExpressionNotSupportedException`` | |
by default with v3.0. To enable client-side projections, set the | |
- Queries that use client-side projections will throw an ``ExpressionNotSupportedException`` error | |
by default. To enable client-side projections, set the |
source/upgrade/v3.txt
Outdated
@@ -97,7 +97,16 @@ Version 3.0 Breaking Changes | |||
To configure logging in your application, see the :ref:`csharp-logging` guide. | |||
|
|||
- The LINQ2 provider has been removed from this version of the driver. | |||
You must use LINQ3 for all LINQ queries. | |||
You must use LINQ3 for all LINQ queries. This change has the following effect on the |
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: since there's only one item in this nested list, it might make more sense to just combine it all into one paragraph? Or add a line break after "You must use LINQ3 for all LINQ queries" and start a new paragraph with "Queries that use client-side projections..."
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
(cherry picked from commit 510f9be)
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-44801
Staging Links
Self-Review Checklist