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

Global Params #241

Merged
merged 4 commits into from
Apr 16, 2024
Merged

Global Params #241

merged 4 commits into from
Apr 16, 2024

Conversation

nhtruong
Copy link
Collaborator

Description

Add global query params and apply them to all operations while merging .

Issues Resolved

closes #167

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock
Copy link
Member

dblock commented Apr 13, 2024

The coverage workflow failed on not finding build/opensearch-openapi.yaml. I suspect that the spec build failed silently, you might want to debug that.

Copy link
Contributor

API specs implemented for 246/649 (37%) APIs.

Signed-off-by: Theo Truong <[email protected]>
Copy link
Contributor

API specs implemented for 246/649 (37%) APIs.

Signed-off-by: Theo Truong <[email protected]>
# updated docs
@nhtruong nhtruong marked this pull request as ready for review April 16, 2024 01:27
Copy link
Contributor

API specs implemented for 246/649 (37%) APIs.

@dblock dblock merged commit 4299a19 into opensearch-project:main Apr 16, 2024
5 checks passed
@nhtruong nhtruong deleted the global_params branch April 16, 2024 15:46
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

Successfully merging this pull request may close these issues.

[BUG] Missing definitions of common/global parameters
3 participants