-
Notifications
You must be signed in to change notification settings - Fork 99
Openapi: adding privileges #4434
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
base: main
Are you sure you want to change the base?
Conversation
validation failing because this adds a new field to the openapi output, will be fixed in case that's how we want to add the information |
My two cents is that of the three options I mentioned in #4303 (comment), "an addendum tacked onto the beginning or end of the operation's "description" field" is likely the best current option since it aligns with what Kibana's doing. Otherwise I think if we went the extension route we'd need a more open-ended structure since the privileges in the Kibana and Cloud API docs would need different options than "cluster-privileges" and "index-privileges". |
d42aa76
to
813a7d3
Compare
@lcawl @georgewallace updated so that privileges are now in the description! please help me with the output format because I'm really not a fan of this one ^^" |
@lcawl curious your thoughts but if we follow suite with what Kibana is doing would be something like
|
Could we format it into more complete phrases? For example:
Or if there's concern about including markdown, then maybe:
|
@lcawl I like the first one. It doesnt necessarily match with what Kibana is doing (Get an agent action status) but I think that is ok as your first one likes much better. The Operation summary name is already an H2, for this sake should it be an h3 instead? I see inconsistencies on the pages now so H2 should be fine but just in case. |
0c34ce9
to
3a5d0ce
Compare
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.
Left some code-improvement suggestions.
3a5d0ce
to
b07c082
Compare
Closes #4303.
There was no desired output in the original issue, so I'm not sure if this is the format expected. Let me know and I'll change it!