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 privilege management #68
Conversation
Thanks for submitting this PR. Will review by the end of tomorrow. |
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 for your contribution @morgo ! Comments inline.
Besides, what about we list all design documents and other related reading resource in a separated section at the end?
Co-authored-by: tison <wander4096@gmail.com>
… into add-privileges-docs
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.
Looks good to me. 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.
Two minor suggestions, otherwise LGTM!
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
Thanks for your collaborations! @bb7133 could you to give this PR a final check? Otherwise I'm going to merge this chapter tomorrow. |
Merging...Thanks for your contribution @morgo ! |
@all-contributors please add @morgo for content |
I've put up a pull request to add @morgo! 🎉 |
* Add privilege management * Update src/understand-tidb/privilege-management.md Co-authored-by: tison <wander4096@gmail.com> * Add feedback from reviewer * Address feedback from reviewers Co-authored-by: tison <wander4096@gmail.com>
What issue does this PR solve?
What is changed:
Adds docs for privilege management.