Skip to content

Commit

Permalink
fix i18n stuff
Browse files Browse the repository at this point in the history
  • Loading branch information
flash1293 committed Mar 4, 2019
1 parent 6a30689 commit e1f891f
Show file tree
Hide file tree
Showing 6 changed files with 25 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -48,11 +48,11 @@ export class DashboardAddPanel extends React.Component {
{
defaultMessage: '{savedObjectName} was added to your dashboard',
values: {
selectedTabName: name,
savedObjectName: name,
},
}
),
'data-test-subj': this.state.selectedTab.toastDataTestSubj,
'data-test-subj': 'addObjectToDashboardSuccess',
});
};

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@

import 'ui/doc_table';

import { i18n } from '@kbn/i18n';
import { EmbeddableFactory } from 'ui/embeddable';
import {
EmbeddableInstanceConfiguration,
Expand All @@ -36,7 +37,9 @@ export class SearchEmbeddableFactory extends EmbeddableFactory {
super({
name: 'search',
savedObjectMetaData: {
name: 'Search',
name: i18n.translate('kbn.discover.savedSearch.savedObjectName', {
defaultMessage: 'Saved Search',
}),
type: 'search',
getIconForSavedObject: () => 'search',
showSavedObject: () => true,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
* under the License.
*/

import { i18n } from '@kbn/i18n';
import chrome from 'ui/chrome';
import { EmbeddableFactory } from 'ui/embeddable';
import { getVisualizeLoader } from 'ui/visualize/loader';
Expand Down Expand Up @@ -45,7 +46,7 @@ export class VisualizeEmbeddableFactory extends EmbeddableFactory<VisualizationA
super({
name: 'visualization',
savedObjectMetaData: {
name: 'Visualization',
name: i18n.translate('kbn.visualize.savedObjectName', { defaultMessage: 'Visualization' }),
type: 'visualization',
getIconForSavedObject: savedObject => {
return (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,13 +66,23 @@ export class SearchSelection extends React.Component<SearchSelectionProps> {
type: 'search',
showSavedObject: () => true,
getIconForSavedObject: () => 'search',
name: 'Search',
name: i18n.translate(
'kbn.visualize.newVisWizard.searchSelection.savedObjectType.search',
{
defaultMessage: 'Saved Search',
}
),
},
{
type: 'index-pattern',
showSavedObject: () => true,
getIconForSavedObject: () => 'indexPatternApp',
name: 'Index Pattern',
name: i18n.translate(
'kbn.visualize.newVisWizard.searchSelection.savedObjectType.indexPattern',
{
defaultMessage: 'Index Pattern',
}
),
},
]}
fixedPageSize={this.fixedPageSize}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -431,6 +431,7 @@ class SavedObjectFinder extends React.Component<SavedObjectFinderProps, SavedObj
const currentSavedObjectMetaData = savedObjectMetaData.find(
metaData => metaData.type === item.type
);
const fullName = `${item.title} (${currentSavedObjectMetaData!.name})`;
const iconType = (
currentSavedObjectMetaData ||
({
Expand All @@ -445,14 +446,11 @@ class SavedObjectFinder extends React.Component<SavedObjectFinderProps, SavedObj
onClick={
onChoose
? () => {
onChoose(
item.id,
item.type,
currentSavedObjectMetaData ? currentSavedObjectMetaData.name : ''
);
onChoose(item.id, item.type, fullName);
}
: undefined
}
title={fullName}
data-test-subj={`savedObjectTitle${(item.title || '').split(' ').join('-')}`}
/>
);
Expand Down
3 changes: 2 additions & 1 deletion test/functional/services/dashboard/add_panel.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ export function DashboardAddPanelProvider({ getService, getPageObjects }) {
await this.filterEmbeddableNames(searchName);

await testSubjects.click(`savedObjectTitle${searchName.split(' ').join('-')}`);
await testSubjects.exists('addSavedSearchToDashboardSuccess');
await testSubjects.exists('addObjectToDashboardSuccess');
await this.closeAddPanel();
}

Expand All @@ -174,6 +174,7 @@ export function DashboardAddPanelProvider({ getService, getPageObjects }) {
await this.toggleFilter('visualization');
await this.filterEmbeddableNames(`"${vizName.replace('-', ' ')}"`);
await testSubjects.click(`savedObjectTitle${vizName.split(' ').join('-')}`);
await testSubjects.exists('addObjectToDashboardSuccess');
await this.closeAddPanel();
}

Expand Down

0 comments on commit e1f891f

Please sign in to comment.