-
Notifications
You must be signed in to change notification settings - Fork 105
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
Doc 355 - client doc updates #1497
Conversation
✅ Deploy Preview for hardcore-allen-f5257d 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.
Thanks @amandalindsay it's looking good - left some specific suggestions that affect a few of the pages
Co-authored-by: Oliver Howell <[email protected]>
Co-authored-by: Oliver Howell <[email protected]>
Co-authored-by: Oliver Howell <[email protected]>
Co-authored-by: Oliver Howell <[email protected]>
Co-authored-by: Oliver Howell <[email protected]>
Co-authored-by: Oliver Howell <[email protected]>
@oliverhowell Please take a look at the tweaks I made as a result of your comments. Hopefully we're good! |
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 @amandalindsay thanks for addressing suggestions
You'll need to change from Draft PR too
Update the individual client doc topics with improved descriptions based on what's already there, plus information from the main HZC website (e.g. https://hazelcast.com/developers/clients/cplusplus/).
Note that the Java client topic will be done in a separate PR (this is to avoid merge conflicts resulting from changes for Doc-294)