-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
script field support, fix field formatters #33910
Changes from all commits
6b1e2e0
e6d7266
63dad94
4145f0d
9598544
2ea3013
1fff2a5
ebf2b38
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -76,12 +76,18 @@ export interface SavedObject { | |
references: SavedObjectReference[]; | ||
} | ||
|
||
/* This object is passed to different helpers in different parts of the code | ||
- packages/kbn-es-query/src/es_query/build_es_query | ||
- x-pack/plugins/reporting/export_types/csv/server/lib/field_format_map | ||
The structure has redundant parts and json-parsed / json-unparsed versions of the same data | ||
*/ | ||
export interface IndexPatternSavedObject { | ||
title: string; | ||
timeFieldName: string; | ||
fields: any[]; | ||
fieldFormatMap: { | ||
[key: string]: { id: string; params: { pattern: string } }; | ||
attributes: { | ||
fieldFormatMap: string; | ||
fields: string; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was some structure tweaks to this type made, after realizing that |
||
}; | ||
} | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,8 +12,7 @@ import { | |
InfraOpsGraphQLProvider | ||
} from './services'; | ||
|
||
export default async function ({ readConfigFile }) { | ||
|
||
export async function getApiIntegrationConfig({ readConfigFile }) { | ||
const kibanaAPITestsConfig = await readConfigFile(require.resolve('../../../test/api_integration/config.js')); | ||
const xPackFunctionalTestsConfig = await readConfigFile(require.resolve('../functional/config.js')); | ||
const kibanaCommonConfig = await readConfigFile(require.resolve('../../../test/common/config.js')); | ||
|
@@ -47,3 +46,5 @@ export default async function ({ readConfigFile }) { | |
esTestCluster: xPackFunctionalTestsConfig.get('esTestCluster'), | ||
}; | ||
} | ||
|
||
export default getApiIntegrationConfig; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just a little trick I saw in another test file to get my test config to "extend" this one in an easier way. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -304,8 +304,6 @@ | |
{ | ||
"type": "index", | ||
"value": { | ||
"aliases": { | ||
}, | ||
"index": "sales", | ||
"mappings": { | ||
"properties": { | ||
|
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.
Nice cleanup here.