-
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
[Discover] shim with local application service #49483
Merged
kertal
merged 219 commits into
elastic:master
from
kertal:kertal-pr-2019-10-24-discover-plugin-shim
Nov 29, 2019
Merged
Changes from 189 commits
Commits
Show all changes
219 commits
Select commit
Hold shift + click to select a range
6046c50
Move doc folder to discover folder
kertal 16a7a6a
Move doc_viewer folder to discover folder
kertal 522db68
Move context folder to discover folder
kertal 6ca152e
Move context scss import to discover folder
kertal 3c762b7
Introduce legacy folder, start migration
kertal fd7e81b
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 1cbd678
Add basic NP structure, structural refactoring
kertal fdbdde0
Migrate discover imports to dependencies.ts
kertal 9fe6e84
Add context placeholder readme
kertal 9ac64ac
Move doc_viewer directive to angular
kertal e65ae78
Move doc_table directive to angular directory
kertal 2af2593
Migrate doc_table dependencies to central file
kertal 09e2c22
Migrate chrome usage to new platform
kertal 62f3164
centralize dependencies
flash1293 0bf00ae
move more stuff into kibana_services
flash1293 e72e51a
fix tests and dependency
flash1293 b42f417
bootstrap home via local application service
flash1293 0d2521e
fix routing for home app
flash1293 12fb70b
Merge remote-tracking branch 'upstream/master' into centralize-home
flash1293 b5aa43c
fix merge conflixts
kertal af3ad7f
switch to dependency getter instead of direct exports
flash1293 a455322
Merge branch 'centralize-home' into local-application-service
flash1293 4424545
Refactor dependencies.ts to kibana_services.ts
kertal a1295d3
clean up
flash1293 082748a
add redirect functionality to local application service
flash1293 2bf3bf6
return early from route change handlers on dummy wrapper route
flash1293 7ee29c6
Improve basic plugin structure
kertal 398299c
fix jest tests
flash1293 44bcd46
Start shimming dashboard
flash1293 0f6c147
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal dbdb5d2
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 001fcd5
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal b0d0a04
Centralize context deps
kertal 822cdac
Merge remote-tracking branch 'upstream/master' into centralize-home
flash1293 103a8f2
Centralize embeddable deps
kertal d824612
Migrate context to angular folder
kertal 4ea2e63
fix broken tests
flash1293 22c5588
Merge branch 'centralize-home' into local-application-service
flash1293 1bd6b3b
local application service
flash1293 ae8dd12
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 fa25ee7
Refactor adding getServices to kibana_services.ts
kertal 162d14d
Replace npStart usage by getService usage
kertal afafe10
wip
flash1293 a7f2826
wip
flash1293 dbcf159
Merge branch 'local-application-service' into shim-dashboard
flash1293 c417060
fix mounting home several times
flash1293 0f2b23b
Merge branch 'local-application-service' into shim-dashboard
flash1293 03fd207
wip
flash1293 40ed5e4
Cleanup / centralize leftover dependencies
kertal 72bfb43
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal ae9ae7e
Fix no_results.test.js
kertal be9fc5a
Fix SCSS import paths
kertal 46a0cca
Restructure kibana_services.ts
kertal de9fcbf
Fix no_results.test.js
kertal 2337329
Move field_name directive back to field_chooser
kertal 77ab276
Fix open_search_panel.test.js
kertal 293365e
Skip doc_viewer tests (unskip later)
kertal 6dc3a71
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 7e88216
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 80351b5
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 8ab13d7
Separate services from stateless dependencies
kertal 889b771
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 7f324d2
unskip doc_viewer.test.tsx
kertal ca5cdb7
Restore the initial way embeddables are registered
kertal 0388c48
Remove TODOs from doc_viewer.test.tsx
kertal 9407bbb
Refactor SEARCH_EMBEDDABLE_TYPE to contants.ts
kertal 66a4651
Fix open_search_panel.test.js
kertal b1e71fe
Change import of SEARCH_EMBEDDABLE_TYPE to fix x-pack test failures
kertal 07287af
Fix doc.test.tsx
kertal 01f815b
Fix doc.test.tsx
kertal 0e8bb19
Add embeddable comments
kertal c8efbe2
Add addBasePath + getInjector to services
kertal 006dbba
Refactor embeddable registration
kertal eea7c62
move handlers from addSetupWork into legacy_compat
flash1293 4f0c5b2
pass dependencies into angular configurator
flash1293 f1ff388
centralize dependencies and clean up shim
flash1293 48ecdf7
Merge branch 'master' into flash1293/local-application-service
elasticmachine a56617d
Implement createEsService function
kertal 05feea4
Adaption of ui/public modules
kertal d1700d2
remove requireDefaultIndex and provide helper function
flash1293 0dd5d3a
rename function and improve documentation
flash1293 e8c8886
Initial version
kertal 02cd426
Adaptions for doc service
kertal 74fb184
Merge remote-tracking branch 'upstream/master' into flash1293/local-a…
flash1293 79a1a14
Merge branch 'flash1293/local-application-service' of github.com:flas…
flash1293 50b1d97
Merge branch 'flash1293/local-application-service' into shim-dashboard
flash1293 ffe6adc
Merging local_application_service changes
kertal 4f9d6a9
Adapting context + doc url routes
kertal 105af0a
redirect unknown urls to default app
flash1293 e0e80c2
Adaptions to render_app.ts
kertal 6decd13
Add promise to provide functional indexPattern resolving
kertal 747ed66
Implement getSavedSearchById as service
kertal f9b47f1
only wire up local angular once
flash1293 d52c6c9
Add $listen and $watchMulti directives
kertal 68b7c4b
context adaptions
kertal a4dc8f5
Add icon directive
kertal bd31c10
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 0777b47
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 e435c7d
Implement forwarding
kertal 1246675
fix top nav and dashboard only mode
flash1293 87c243b
Add even more missing directives
kertal 3cf5fe4
Refactor collabsible_sidebar.js
kertal 55ebcb0
Refactor debounce.js
kertal 91723c5
Refactor fixed_scroll.js
kertal 4ce4a06
Refactor css_truncate.js
kertal 253045f
Implement getAngularDependencies function
kertal 44b4303
Refactor code to use getAngularDependencies function
kertal cf087b8
Shim filterbar directive + depended directives
kertal a34dd9f
decouple frome home shim PR
flash1293 d1defcc
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 e8c59f7
fix typing problems
flash1293 e238a5d
fix other bugs
flash1293 4e76a5e
merge upstream/master
kertal 3ef6254
got rid of two other angular dependencies
flash1293 ce902cd
fix i18n issue
flash1293 8022d4d
fix various issues
flash1293 936f09a
added some debug notes
flash1293 784a20e
fix vis filters
flash1293 ba78539
improve
flash1293 ac462b5
fix hooks handling for local routes in legacy_compat
flash1293 72b52ae
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 3c619d2
fix embedded visualize handler tests
flash1293 02caf27
fix test failures
flash1293 144699b
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 78855e1
remove buggy test
flash1293 14d8fd4
different take on global state handling
flash1293 fa13e4b
fix detection of url state
flash1293 2a5c0fb
fix broken test
flash1293 08c8740
Fix reporting imports to prevent loading the whole embeddable
kertal f5599ab
Fix tests and embeddables
kertal a47e412
Fix types
kertal 0cb6533
Add render-complete directive
kertal d5acbd3
Fix functional test in firefox
kertal 45e875d
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 5f9120b
never update state in dummy route mode
flash1293 8079a3f
Fix jest test
kertal 2d868f2
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 296c174
clean up implementation
flash1293 8fb0fbd
fix type error
flash1293 c1adec3
Fix eventually undefined searchScore at search_embeddable
kertal a82bd86
Fix mocha test
kertal af1ed6c
Merge and adapt joe changes
kertal bf5a71e
Merge branch 'flash1293-shim-dashboard' into kertal-pr-2019-10-24-dis…
kertal 1f6329f
Fix unresolved dependencies
kertal fff216c
Merge branch 'flash1293-shim-dashboard' into kertal-pr-2019-10-24-dis…
kertal 42a0d8b
Fix breadcrumbs
kertal d93bb6c
Refactor to use ensureDefaultIndexPattern
kertal f5ebcf7
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 fad42dd
remove session storage global state handling for now
flash1293 0f395c7
Fix tests
kertal f954975
Fix context test
kertal 39ce528
remove session storage global state handling for now
flash1293 6d3b63c
Merge branch 'flash1293-shim-dashboard' into kertal-pr-2019-10-24-dis…
kertal f6d4e75
Bootstrap separate angular module for embeddables first draft
kertal f005b4e
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 dcb9a88
remove unused dependencies
flash1293 01edc51
Slim embeddable angular
kertal a21ba30
started centralizing and cleaning up imports
flash1293 d0c4817
Refactor queryFilter in embeddable
kertal 7ec261d
centralize external imports
flash1293 aaf0a8e
Invoke plugin async
kertal 168001b
Revert chromedriver update (#50324)
flash1293 928f34f
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 a0abd00
fix import bugs
flash1293 42db084
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 8bdca65
fix merge bugs and rename main dynamic entrypoint
flash1293 042f2e8
fix jest tests
flash1293 363a907
fix saved object finder bug
flash1293 a0c72cf
Fix unit test, refactor angular bootstrapping to be done when needed
kertal 9296c4c
revert using stateless component for this PR
flash1293 259cc5d
Adapt even more unit tests to bootstrap the local angular just in case
kertal 8d41818
fix types
flash1293 44c15d7
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 a4dd4ff
revert unrelated changes and improve implementation
flash1293 d51dcd5
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 f3e10c7
merge shim-dashboard
kertal 90bf3aa
Cleanup
kertal 08d709b
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 ac541f2
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 3448cfb
Refactoring of services + better typescript types
kertal e8a6d95
Change the way isEditable is used in embeddable_factory (caused failure)
kertal 819b127
Don't catch if setServices is called twice (2 embeddables rendered pa…
kertal 0ccd9ad
Restructure and add comments
kertal 314f78a
fix jest tests
kertal c1ccc16
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 c4c5bc2
review fixes
flash1293 4b873e7
fix functional tests
flash1293 1b20af9
fix dashboard icon
flash1293 4718578
merge shim-dashboard/fix conflicts
kertal e780906
Fix missing filter bar in context view
kertal 75904ed
Fix isLoading state in embeddable rendering
kertal 8078022
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal 2ea9238
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 c445d72
remove bad import and do not deep-import types from data plugin
flash1293 6c4e756
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 6f4bbdb
Merge remote-tracking branch 'upstream/master' into shim-dashboard
flash1293 78fd82c
Fix invalid share url
kertal 1dc39eb
merge shim-dashboard
kertal 6e770a6
Add missing lib for unit testing
kertal b5c86b5
improve types and remove unused imports
kertal 91fd89f
merge upstream/master
kertal 6dda3ae
merge upstream/master
kertal ad13c2e
Add some comments
kertal 28cc1fe
Review improvements
kertal 298fac9
Implement use of FilterStateManger to solve state <-> url syncing
kertal 48257f1
Merge remote-tracking branch 'upstream/master' into kertal-pr-2019-10…
kertal ea1755c
Refactor to replace localApplicationService
kertal e4cf0ee
Cleanup filterStateManager when scope is destroyed
kertal 7511ed7
Remove unnecessary ts-ignore
kertal c04f54a
Tiny type changes
kertal cf91768
Remove missing `searchScope` warning for embeddables
kertal 83f1bb4
Address review feedback
kertal 2ff88db
Improve app mountpoint, remove innerHTML
kertal d466df1
Improve types
kertal b33c13a
Add findByCssSelector to ensure the charts have been rendered
kertal 9aae0d1
Merge remote-tracking branch upstream/master into kertal-pr-2019-10-2…
kertal 088b56e
Fix type error
kertal c7bdc8a
Merge conflicts
kertal d055d83
Fix merge conflicts
kertal File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 changes: 1 addition & 1 deletion
2
src/legacy/core_plugins/kibana/public/dashboard/_dashboard_app.scss
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
.dshAppContainer { | ||
flex: 1; | ||
display: flex; | ||
flex-direction: column; | ||
height: 100%; | ||
} | ||
|
||
.dshStartScreen { | ||
|
220 changes: 220 additions & 0 deletions
220
src/legacy/core_plugins/kibana/public/dashboard/application.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,220 @@ | ||
/* | ||
* Licensed to Elasticsearch B.V. under one or more contributor | ||
* license agreements. See the NOTICE file distributed with | ||
* this work for additional information regarding copyright | ||
* ownership. Elasticsearch B.V. licenses this file to you under | ||
* the Apache License, Version 2.0 (the "License"); you may | ||
* not use this file except in compliance with the License. | ||
* You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, | ||
* software distributed under the License is distributed on an | ||
* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY | ||
* KIND, either express or implied. See the License for the | ||
* specific language governing permissions and limitations | ||
* under the License. | ||
*/ | ||
|
||
import { EuiConfirmModal } from '@elastic/eui'; | ||
import angular, { IModule } from 'angular'; | ||
import { IPrivate } from 'ui/private'; | ||
import { i18nDirective, i18nFilter, I18nProvider } from '@kbn/i18n/angular'; | ||
import { | ||
AppMountContext, | ||
ChromeStart, | ||
LegacyCoreStart, | ||
SavedObjectsClientContract, | ||
UiSettingsClientContract, | ||
} from 'kibana/public'; | ||
import { Storage } from '../../../../../plugins/kibana_utils/public'; | ||
import { | ||
GlobalStateProvider, | ||
StateManagementConfigProvider, | ||
AppStateProvider, | ||
PrivateProvider, | ||
EventsProvider, | ||
PersistedState, | ||
createTopNavDirective, | ||
createTopNavHelper, | ||
PromiseServiceCreator, | ||
KbnUrlProvider, | ||
RedirectWhenMissingProvider, | ||
confirmModalFactory, | ||
configureAppAngularModule, | ||
} from './legacy_imports'; | ||
|
||
// @ts-ignore | ||
import { initDashboardApp } from './legacy_app'; | ||
import { DataStart } from '../../../data/public'; | ||
import { SavedQueryService } from '../../../data/public/search/search_bar/lib/saved_query_service'; | ||
import { EmbeddablePublicPlugin } from '../../../../../plugins/embeddable/public'; | ||
import { NavigationStart } from '../../../navigation/public'; | ||
import { DataPublicPluginStart as NpDataStart } from '../../../../../plugins/data/public'; | ||
|
||
export interface RenderDeps { | ||
core: LegacyCoreStart; | ||
indexPatterns: DataStart['indexPatterns']['indexPatterns']; | ||
dataStart: DataStart; | ||
npDataStart: NpDataStart; | ||
navigation: NavigationStart; | ||
shareContextMenuExtensions: any; | ||
savedObjectsClient: SavedObjectsClientContract; | ||
savedObjectRegistry: any; | ||
dashboardConfig: any; | ||
savedDashboards: any; | ||
dashboardCapabilities: any; | ||
uiSettings: UiSettingsClientContract; | ||
chrome: ChromeStart; | ||
addBasePath: (path: string) => string; | ||
savedQueryService: SavedQueryService; | ||
embeddables: ReturnType<EmbeddablePublicPlugin['start']>; | ||
localStorage: Storage; | ||
} | ||
|
||
let angularModuleInstance: IModule | null = null; | ||
|
||
export const renderApp = (element: HTMLElement, appBasePath: string, deps: RenderDeps) => { | ||
if (!angularModuleInstance) { | ||
angularModuleInstance = createLocalAngularModule(deps.core, deps.navigation); | ||
// global routing stuff | ||
configureAppAngularModule(angularModuleInstance, deps.core as LegacyCoreStart, true); | ||
// custom routing stuff | ||
initDashboardApp(angularModuleInstance, deps); | ||
} | ||
const $injector = mountDashboardApp(appBasePath, element); | ||
return () => { | ||
$injector.get('$rootScope').$destroy(); | ||
}; | ||
}; | ||
|
||
const mainTemplate = (basePath: string) => `<div style="height: 100%"> | ||
<base href="${basePath}" /> | ||
<div ng-view style="height: 100%;"></div> | ||
</div> | ||
`; | ||
|
||
const moduleName = 'app/dashboard'; | ||
|
||
const thirdPartyAngularDependencies = ['ngSanitize', 'ngRoute', 'react']; | ||
|
||
function mountDashboardApp(appBasePath: string, element: HTMLElement) { | ||
const mountpoint = document.createElement('div'); | ||
mountpoint.setAttribute('style', 'height: 100%'); | ||
// eslint-disable-next-line | ||
mountpoint.innerHTML = mainTemplate(appBasePath); | ||
// bootstrap angular into detached element and attach it later to | ||
// make angular-within-angular possible | ||
const $injector = angular.bootstrap(mountpoint, [moduleName]); | ||
// initialize global state handler | ||
element.appendChild(mountpoint); | ||
return $injector; | ||
} | ||
|
||
function createLocalAngularModule(core: AppMountContext['core'], navigation: NavigationStart) { | ||
createLocalI18nModule(); | ||
createLocalPrivateModule(); | ||
createLocalPromiseModule(); | ||
createLocalConfigModule(core); | ||
createLocalKbnUrlModule(); | ||
createLocalStateModule(); | ||
createLocalPersistedStateModule(); | ||
createLocalTopNavModule(navigation); | ||
createLocalConfirmModalModule(); | ||
|
||
const dashboardAngularModule = angular.module(moduleName, [ | ||
...thirdPartyAngularDependencies, | ||
'app/dashboard/Config', | ||
'app/dashboard/I18n', | ||
'app/dashboard/Private', | ||
'app/dashboard/PersistedState', | ||
'app/dashboard/TopNav', | ||
'app/dashboard/State', | ||
'app/dashboard/ConfirmModal', | ||
]); | ||
return dashboardAngularModule; | ||
} | ||
|
||
function createLocalConfirmModalModule() { | ||
angular | ||
.module('app/dashboard/ConfirmModal', ['react']) | ||
.factory('confirmModal', confirmModalFactory) | ||
.directive('confirmModal', reactDirective => reactDirective(EuiConfirmModal)); | ||
} | ||
|
||
function createLocalStateModule() { | ||
angular | ||
.module('app/dashboard/State', [ | ||
'app/dashboard/Private', | ||
'app/dashboard/Config', | ||
'app/dashboard/KbnUrl', | ||
'app/dashboard/Promise', | ||
'app/dashboard/PersistedState', | ||
]) | ||
.factory('AppState', function(Private: any) { | ||
return Private(AppStateProvider); | ||
}) | ||
.service('getAppState', function(Private: any) { | ||
return Private(AppStateProvider).getAppState; | ||
}) | ||
.service('globalState', function(Private: any) { | ||
return Private(GlobalStateProvider); | ||
}); | ||
} | ||
|
||
function createLocalPersistedStateModule() { | ||
angular | ||
.module('app/dashboard/PersistedState', ['app/dashboard/Private', 'app/dashboard/Promise']) | ||
.factory('PersistedState', (Private: IPrivate) => { | ||
const Events = Private(EventsProvider); | ||
return class AngularPersistedState extends PersistedState { | ||
constructor(value: any, path: any) { | ||
super(value, path, Events); | ||
} | ||
}; | ||
}); | ||
} | ||
|
||
function createLocalKbnUrlModule() { | ||
angular | ||
.module('app/dashboard/KbnUrl', ['app/dashboard/Private', 'ngRoute']) | ||
.service('kbnUrl', (Private: IPrivate) => Private(KbnUrlProvider)) | ||
.service('redirectWhenMissing', (Private: IPrivate) => Private(RedirectWhenMissingProvider)); | ||
} | ||
|
||
function createLocalConfigModule(core: AppMountContext['core']) { | ||
angular | ||
.module('app/dashboard/Config', ['app/dashboard/Private']) | ||
.provider('stateManagementConfig', StateManagementConfigProvider) | ||
.provider('config', () => { | ||
return { | ||
$get: () => ({ | ||
get: core.uiSettings.get.bind(core.uiSettings), | ||
}), | ||
}; | ||
}); | ||
} | ||
|
||
function createLocalPromiseModule() { | ||
angular.module('app/dashboard/Promise', []).service('Promise', PromiseServiceCreator); | ||
} | ||
|
||
function createLocalPrivateModule() { | ||
angular.module('app/dashboard/Private', []).provider('Private', PrivateProvider); | ||
} | ||
|
||
function createLocalTopNavModule(navigation: NavigationStart) { | ||
angular | ||
.module('app/dashboard/TopNav', ['react']) | ||
.directive('kbnTopNav', createTopNavDirective) | ||
.directive('kbnTopNavHelper', createTopNavHelper(navigation.ui)); | ||
} | ||
|
||
function createLocalI18nModule() { | ||
angular | ||
.module('app/dashboard/I18n', []) | ||
.provider('i18n', I18nProvider) | ||
.filter('i18n', i18nFilter) | ||
.directive('i18nId', i18nDirective); | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
👍 FWIW, I see this same change was tested and approved in Joe's PR - https://github.com/elastic/kibana/pull/48913/files#diff-3edabc97f406f1537759321bb012ec14R4