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

[Draft] Views documentation #6394

Closed
wants to merge 2 commits into from
Closed

Conversation

peternied
Copy link
Member

Description

Adds documentation for the views feature

Issues Resolved

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@stephen-crawford
Copy link
Contributor

@peternied maybe I missed it, but could you add a really clear explanation about the interaction between Views and roles? I think we want to make it really clear that we still use the normal permission system behind the scenes otherwise it may seem like Views can skip the normal check or are an alternative permission system.

@peternied
Copy link
Member Author

@peternied maybe I missed it, but could you add a really clear explanation about the interaction between Views and roles? I think we want to make it really clear that we still use the normal permission system behind the scenes otherwise it may seem like Views can skip the normal check or are an alternative permission system.

I have documented the feature as it is, as it changes I'll need to revise this PR.

There is no change or loosening to any permissions - you'll need both index permissions as normal in addition to cluster permissions that allow calling the different view APIs

Signed-off-by: Peter Nied <[email protected]>
@peternied
Copy link
Member Author

I'm going to close this out - until there is a clear delivery data associated with this feature it doesn't make sense to clog up the PR queue. I'll reopen if it becomes relevant in the future.

@peternied peternied closed this Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Views
2 participants