-
Notifications
You must be signed in to change notification settings - Fork 15
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
Propose a standard for OAS operationIDs. #208
Comments
The operationID is a great place to add information that could be relevant to generated clients. I think we should consider how much information we could inject into it before finalizing it. |
@mkistler raised that operationIDs are used by tools like the openapi-generator. I tried out the generator, and it looks like the mapping looks something like a direct conversion, with delimiters such as I modified aepc so that the operation IDs match the grpc RPC name, and the result in the client looks pretty good:
So at the moment I am proposing:
This:
|
To help with more standard generation of clients, propose a standard for OAS operation IDs. fixes aep-dev#208
OpenAPI specifications have an "operationID" field, of which there is no prescriptive syntax.
We should try define a standard for AEPs that will serve the clients.
Context:
🤔 well, I don't know if we've actually made a decision about the operationId yet, and I think that it's usage will be very critical when we start building clients.
For now I'll remove it, and will probably do a follow-up to determine the standard around OAS operations IDs.
Originally posted by @toumorokoshi in #206 (comment)
The text was updated successfully, but these errors were encountered: