-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add handlers for read, update, and delete to CREATE_YOUR_FIRST_EXTENSION doc #864
Conversation
Signed-off-by: Daniel Widdis <widdis@gmail.com>
Signed-off-by: Daniel Widdis <widdis@gmail.com>
Signed-off-by: Daniel Widdis <widdis@gmail.com>
Signed-off-by: Daniel Widdis <widdis@gmail.com>
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.
A couple of comments, mainly making the document more user-centric. Thanks!
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.
A few more comments, some similar to @kolchfa-aws but with an addition or two. Thanks!
Thanks! Can you tell I spent way too long in academia where "we" is the standard? :) |
Signed-off-by: Daniel Widdis <widdis@gmail.com>
Yes, "we" when we're doing an experiment, and "you" when you want the user to do something :) |
@kolchfa-aws @natebower looks good to you before we merge the changes in? |
LGTM 😄 |
…ION doc (#864) * Add handlers for read, update, and delete Signed-off-by: Daniel Widdis <widdis@gmail.com> * Remove reference to SDKClient Signed-off-by: Daniel Widdis <widdis@gmail.com> * Add link to TOC Signed-off-by: Daniel Widdis <widdis@gmail.com> * Replace we with you and only use one space after a full stop Signed-off-by: Daniel Widdis <widdis@gmail.com> --------- Signed-off-by: Daniel Widdis <widdis@gmail.com> Co-authored-by: Owais Kazi <owaiskazi19@gmail.com> (cherry picked from commit 67046b2) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ION doc (#864) (#878) * Add handlers for read, update, and delete * Remove reference to SDKClient * Add link to TOC * Replace we with you and only use one space after a full stop --------- (cherry picked from commit 67046b2) Signed-off-by: Daniel Widdis <widdis@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Owais Kazi <owaiskazi19@gmail.com>
…ION doc (opensearch-project#864) (opensearch-project#878) * Add handlers for read, update, and delete * Remove reference to SDKClient * Add link to TOC * Replace we with you and only use one space after a full stop --------- (cherry picked from commit 67046b2) Signed-off-by: Daniel Widdis <widdis@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Owais Kazi <owaiskazi19@gmail.com>
…ION doc (opensearch-project#864) (opensearch-project#878) * Add handlers for read, update, and delete * Remove reference to SDKClient * Add link to TOC * Replace we with you and only use one space after a full stop --------- (cherry picked from commit 67046b2) Signed-off-by: Daniel Widdis <widdis@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Owais Kazi <owaiskazi19@gmail.com>
Description
Updates
CREATE_YOUR_FIRST_EXTENSION
with the RUD part of CRUD (and a better C example). Also fixes the internal links broken in earlier reviews.Please merge #827 (and then rebase this PR) prior to merging this PR, as this builds on updates from that PR to Named Routes.
Reference project implementing this, with the code tested: https://github.com/dbwiddis/CRUDExtension
Issues Resolved
Part of #836 (Documentation)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.