-
Notifications
You must be signed in to change notification settings - Fork 25
DOCSP-50559: query guids #646
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-50559: query guids #646
Conversation
✅ Deploy Preview for docs-csharp ready!
To edit notification comments on pull requests, go to your Netlify project 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!
I don't think we should have a separate section for "Query GUIDs". The users will then expect an example that includes Builders and LINQ as well along with the BSON based example mentioned there. Just a simple callout note on how to construct/work with legacy GUIDs should be good. The Upgrade guide can then just link to that callout. |
The BSONBinaryData.Create example in the v3 upgrade guide is applicable to the new GUIDs as well. I suggest that we should change the text to "To view an example for using legacy GUIDs, see the Construct Legacy GUIDs note in the GUID Serialization guide." That ways we are being explicit. |
* DOCSP-50559: query guids * RB restructure * RB small fix (cherry picked from commit 004a229)
* DOCSP-50559: query guids * RB restructure * RB small fix (cherry picked from commit 004a229)
* DOCSP-50559: query guids * RB restructure * RB small fix (cherry picked from commit 004a229)
* DOCSP-50559: query guids * RB restructure * RB small fix (cherry picked from commit 004a229)
* DOCSP-50559: query guids * RB restructure * RB small fix (cherry picked from commit 004a229)
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-50559
Staging Links
Self-Review Checklist