-
Notifications
You must be signed in to change notification settings - Fork 3.5k
[8.19] (backport #17854) adding x-metatags to openapi spec #18011
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
base: 8.19
Are you sure you want to change the base?
Conversation
(cherry picked from commit bc4645c) # Conflicts: # docs/static/spec/openapi/logstash-api.yaml
Cherry-pick of bc4645c has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
📃 DOCS PREVIEW ✨ https://logstash_bk_18011.docs-preview.app.elstc.co/diff |
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
6 similar comments
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
This pull request has not been merged yet. Could you please review and merge it @georgewallace? 🙏 |
Release notes
What does this PR do?
This PR updates the OpenAPI spec to add the x-metatags extension. This extension adds product_name metadata which during docs build time is converted to a meta tag on the HTML page, this tag is then parsed by our internal site search to add the API docs pages to the product facets on search.
Why is it important/What is the impact to the user?
This allows users to filter their search by logstash and only see logstash apis
Checklist
Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs
This is an automatic backport of pull request #17854 done by [Mergify](https://mergify.com).