Skip to content
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

docs: update Kafka ACL docs #1931

Merged
merged 1 commit into from
Dec 11, 2024
Merged

docs: update Kafka ACL docs #1931

merged 1 commit into from
Dec 11, 2024

Conversation

staceysalamon-aiven
Copy link
Contributor

About this change—what it does

Updates descriptions for Aiven ACLs and Kafka-native ACLs to provide clarification on when to use each. Adds example usage for Kafka-native ACLs. Updates field descriptions for native ACLs.

@staceysalamon-aiven staceysalamon-aiven added documentation Improvements or additions to documentation no changelog No changelog entries are required for this PR labels Dec 6, 2024
@staceysalamon-aiven staceysalamon-aiven marked this pull request as ready for review December 9, 2024 16:09
@staceysalamon-aiven staceysalamon-aiven requested a review from a team as a code owner December 9, 2024 16:09
roope-kar
roope-kar previously approved these changes Dec 10, 2024
Copy link
Member

@roope-kar roope-kar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, @harshini-rangaswamy do you want to take a look?

@harshini-rangaswamy
Copy link

lgtm, @harshini-rangaswamy do you want to take a look?

It looks good to me, too. Thanks, @staceysalamon-aiven!

@staceysalamon-aiven
Copy link
Contributor Author

lgtm, @harshini-rangaswamy do you want to take a look?

It looks good to me, too. Thanks, @staceysalamon-aiven!

Thanks for your help!

Updates descriptions for Aiven ACLs and Kafka-native ACLs
to provide clarification on when to use each. Adds example
usage for Kafka-native ACLs. Updates field descriptions for
native ACLs.
@byashimov byashimov merged commit 14fe2c1 into main Dec 11, 2024
10 checks passed
@byashimov byashimov deleted the staceys-doc-1247 branch December 11, 2024 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation no changelog No changelog entries are required for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants