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

[main-lts] remove operation id prefix #880

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Backport: #873

Note: CI is not automatically triggered on backported PRs, please comment 'ok to test' to launch Jenkins jobs

  • You have read the contributors guide
  • Your code is properly formatted according to our code style
  • Pull Request title contains the target branch if not targeting main: [0.9.x] Subject
  • Pull Request contains link to the issue
  • Pull Request contains link to any dependent or related Pull Request
  • Pull Request contains description of the issue
  • Pull Request does not include fixes for issues other than the main ticket

Fix #866

When use smallrye openapi operationid strategy CLASS_METHOD,operationId fields take the form of "class_name_method". For example, "User_findAll". We want remove operationId prefix (e.g. User_findAll=> findAll)

In this PR, we add a config properties remove-operation-id-prefix,to delete class name from operationId. The properties are passed to the openapi-generator, and the specific functions are executed by the openapi-generator.

https://github.com/OpenAPITools/openapi-generator/blob/4c5a57fe7bae59483bb8fa9218be8dc4f82e1bda/modules/openapi-generator/src/main/java/org/openapitools/codegen/DefaultCodegen.java#L5638

@github-actions github-actions bot requested a review from a team as a code owner November 29, 2024 15:48
@hbelmiro hbelmiro closed this Nov 29, 2024
@hbelmiro hbelmiro reopened this Nov 29, 2024
Copy link
Contributor Author

🎊 PR Preview 2711260 has been successfully built and deployed. See the documentation preview: https://quarkus-openapi-generator-preview-pr-880.surge.sh

@hbelmiro hbelmiro merged commit 10be4d3 into main-lts Nov 29, 2024
13 checks passed
@hbelmiro hbelmiro deleted the main-lts_remove-operation-id-prefix branch November 29, 2024 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants