-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Docs] remove Kibana book-scoped attributes from 7.17 #178759
[Docs] remove Kibana book-scoped attributes from 7.17 #178759
Conversation
2628b2b
to
294b0ae
Compare
@@ -6,7 +6,7 @@ We use functional tests to make sure the {kib} UI works as expected. It replaces | |||
[discrete] | |||
=== Running functional tests | |||
|
|||
The `FunctionalTestRunner` is very bare bones and gets most of its functionality from its config file, located at {blob}test/functional/config.js[test/functional/config.js] or {blob}x-pack/test/functional/config.js[x-pack/test/functional/config.js]. If you’re writing a plugin outside the {kib} repo, you will have your own config file. | |||
The `FunctionalTestRunner` is very bare bones and gets most of its functionality from its config file. If you’re writing a plugin outside the {kib} repo, you will have your own config file. |
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.
took out the links to match 8.14 because the links were dead - this file is referenced/linked to in the doc linked lower on the page
@@ -64,7 +64,8 @@ export default async function ({ readConfigFile }) { | |||
} | |||
|
|||
// more settings, like timeouts, mochaOpts, etc are | |||
// defined in the config schema. See {blob}src/functional_test_runner/lib/config/schema.js[src/functional_test_runner/lib/config/schema.js] | |||
// defined in the config schema. | |||
// See {kibana-blob}packages/kbn-test/src/functional_test_runner/lib/config/schema.ts |
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.
@shainaraskas follow up on this path change. need to also port this change forward to main
file was moved here
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.
Confirmed with the Kibana team that this new file is the correct replacement
Pinging @elastic/kibana-docs (Team:Docs) |
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.
LGTM
Fixed in elastic/docs#2963 |
Summary
Removes book-scoped docs attributes from 7.17 in preparation for eventual migration to Docsmobile
backports changes from #178676 and #155210
blob
attribute is replaced by the globalkibana-blob
attributedata-source
anddata-sources
are also present in the global attributesnote: looks like
data-source
anddata-sources
are not respecting the capitalization of the attribute so it's getting overridden by legacy attr. definitions, leading to a capitalization bug. prob should be fixed in the global attributes filehttps://github.com/elastic/kibana/blob/7.17/docs/user/dashboard/dashboard.asciidoc?plain=1#L380
![image](https://private-user-images.githubusercontent.com/58563081/312934370-18a6eaa9-afe1-4057-abaa-9c912f62af89.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjI3MTkwMzAsIm5iZiI6MTcyMjcxODczMCwicGF0aCI6Ii81ODU2MzA4MS8zMTI5MzQzNzAtMThhNmVhYTktYWZlMS00MDU3LWFiYWEtOWM5MTJmNjJhZjg5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MDMlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODAzVDIwNTg1MFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTliNzRkZjgyNGE4NGMzMDdiMjU5MTkzZjBiNGRjYjFmZWU1Zjg5MGUwOGFkYjc3OTYyM2UzODMyNjFmZjZhNGYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.p20Lvc6miTFYcIOWVRb6NJnGxid9woHuVBiqbqvVRrc)