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

[Swagger-API] Fix operationIds in swagger description #6

Open
tlindener opened this issue Oct 28, 2016 · 2 comments
Open

[Swagger-API] Fix operationIds in swagger description #6

tlindener opened this issue Oct 28, 2016 · 2 comments

Comments

@tlindener
Copy link

Hi, the operationIds used in the swagger definition aren't working well with automated code generation. It would be great if you could provide an upgraded version where the ids don't have whitespaces.

@tadhgpearson
Copy link
Collaborator

Thanks for the feedback Thomas.

Our API catalog documentation is generated from this swagger file using Apigee Smartdocs. Today, Smartdocs uses the Operation ID to generate the page title and the left-hand navigation menu, so we have to format the operationID field in the way we want the page titles to be displayed on our site. Since the Smartdocs is closed source, we can't change that.

So I agree that the operation ID should be a programmable identifiers, and the summary should be used for the page title, as is now defined in the spec... but we're forced to work within the limitations of our tooling. I'll raise this issue to the Smartdocs team, and get back to you.

@tlindener
Copy link
Author

Hi,

thank you for the feedback. I understand the problem although I would suggest to change it anyway. The "pain" caused by not so nice headlines is a lot less than the pain caused by the failures caused by wrong operationIds.

Best regards
Tobias

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

No branches or pull requests

2 participants