-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix: change name of the API #210
Conversation
CC @MikeEdgar |
CC @biswassri |
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.
@wtrocki I don't think this change is enough for what you want to do. The OpenAPI file should be regenerated and checked in typically.
OpenAPi file is regenerated - I wanted to avoid conflicts. Noticed that we were missing docs for generation so added that to Readme.md |
Found it. Ignore me for now |
Should to do the work |
kafka-admin/src/main/java/org/bf2/admin/kafka/admin/handlers/OASModelFilter.java
Outdated
Show resolved
Hide resolved
🙏 🙈 |
Content, UX and API team trying to unify api names. All instance apis should have common names to help users to understand architecture.
Small change. Mostly harmless - it will only affect SDK comments.