-
Notifications
You must be signed in to change notification settings - Fork 10
DOCSP-51324: Network Compression #110
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
Conversation
✅ Deploy Preview for docs-kotlin-sync ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
🔄 Deploy Preview for docs-kotlin-sync processing
|
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.
Very nice! Left a few suggestions/comments (this is my first solo bigger PR review so may not be super accurate)
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!
Looks good! I noticed that the existing Java sync page and this Kotlin sync one differ in content, with the Kotlin version providing more detail. Do we have plans to update the Java sync page as well to align the content (aside from the examples)? |
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-51324
Staging Links
Self-Review Checklist