Skip to content
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.

Sign up for GitHub

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

[Remote clusters] Update copy #60382

Merged
merged 4 commits into from
Mar 19, 2020
Merged

[Remote clusters] Update copy #60382

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
79e41e3
address copy feedback
alisonelizabeth Mar 17, 2020
4d28412
address review feedback + update jest snapshot
alisonelizabeth Mar 18, 2020
5e6afb7
Merge branch 'master' of github.com:elastic/kibana into remote_cluste…
alisonelizabeth Mar 18, 2020
72649df
update server_name help text
alisonelizabeth Mar 18, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
37 changes: 6 additions & 31 deletions ...lusters/public/application/sections/components/remote_cluster_form/remote_cluster_form.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,7 @@ import {
htmlIdGenerator,
} from '@elastic/eui';

import {
skippingDisconnectedClustersUrl,
transportPortUrl,
proxyModeUrl,
} from '../../../services/documentation';
import { skippingDisconnectedClustersUrl, transportPortUrl } from '../../../services/documentation';

import { RequestFlyout } from './request_flyout';

Expand Down Expand Up @@ -456,33 +452,14 @@ export class RemoteClusterForm extends Component {
<>
<FormattedMessage
id="xpack.remoteClusters.remoteClusterForm.sectionModeDescription"
defaultMessage="Remote cluster connections work by configuring a remote cluster and connecting only to a limited number of nodes in that remote cluster."
defaultMessage="Use seed nodes by default, or switch to a single proxy address."
/>
<EuiFormRow
hasEmptyLabelSpace
fullWidth
helpText={
<FormattedMessage
id="xpack.remoteClusters.remoteClusterForm.fieldModeDescription"
defaultMessage="Configure a remote cluster with a single proxy address. {learnMoreLink}"
values={{
learnMoreLink: (
<EuiLink href={proxyModeUrl} target="_blank">
<FormattedMessage
id="xpack.remoteClusters.remoteClusterForm.fieldModeDescription.learnMoreLinkLabel"
defaultMessage="Learn more."
/>
</EuiLink>
),
}}
/>
}
>
<EuiFormRow hasEmptyLabelSpace fullWidth>
<EuiSwitch
label={
<FormattedMessage
id="xpack.remoteClusters.remoteClusterForm.fieldModeLabel"
defaultMessage="Use proxy mode"
defaultMessage="Use a single proxy address"
/>
}
checked={mode === PROXY_MODE}
Expand Down Expand Up @@ -523,9 +500,7 @@ export class RemoteClusterForm extends Component {
<p>
<FormattedMessage
id="xpack.remoteClusters.remoteClusterForm.sectionSkipUnavailableDescription"
defaultMessage="By default, a request fails if any of the queried remote clusters
are unavailable. To continue sending a request to other remote clusters if this
cluster is unavailable, enable {optionName}. {learnMoreLink}"
defaultMessage="A request fails if any of the queried remote clusters are unavailable. To send requests to other remote clusters if this cluster is unavailable, enable {optionName}. {learnMoreLink}"
values={{
optionName: (
<strong>
Expand Down Expand Up @@ -839,7 +814,7 @@ export class RemoteClusterForm extends Component {
description={
<FormattedMessage
id="xpack.remoteClusters.remoteClusterForm.sectionNameDescription"
defaultMessage="A unique name for the remote cluster."
defaultMessage="A unique name for the cluster."
/>
}
fullWidth
Expand Down
24 changes: 18 additions & 6 deletions ...ic/application/sections/components/remote_cluster_page_title/remote_cluster_page_title.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,23 +7,22 @@
import React, { Fragment } from 'react';
import PropTypes from 'prop-types';
import { FormattedMessage } from '@kbn/i18n/react';
import { remoteClustersUrl } from '../../../services/documentation';

import {
EuiButtonEmpty,
EuiFlexGroup,
EuiFlexItem,
EuiPageContentHeader,
EuiSpacer,
EuiText,
EuiTitle,
} from '@elastic/eui';

import { remoteClustersUrl } from '../../../services/documentation';

export const RemoteClusterPageTitle = ({ title }) => (
export const RemoteClusterPageTitle = ({ title, description }) => (
<Fragment>
<EuiSpacer size="xs" />

<EuiPageContentHeader>
<EuiTitle size="l">
<EuiFlexGroup justifyContent="spaceBetween" alignItems="center">
<EuiFlexItem grow={false}>
<EuiTitle size="l" data-test-subj="remoteClusterPageTitle">
Expand All @@ -47,10 +46,23 @@ export const RemoteClusterPageTitle = ({ title }) => (
</EuiButtonEmpty>
</EuiFlexItem>
</EuiFlexGroup>
</EuiPageContentHeader>
</EuiTitle>

{description ? (
<>
<EuiSpacer size="s" />

<EuiTitle size="s">
<EuiText color="subdued">{description}</EuiText>
</EuiTitle>
</>
) : null}

<EuiSpacer size="m" />
</Fragment>
);

RemoteClusterPageTitle.propTypes = {
title: PropTypes.node.isRequired,
description: PropTypes.node,
};
6 changes: 6 additions & 0 deletions ...gins/remote_clusters/public/application/sections/remote_cluster_add/remote_cluster_add.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,12 @@ export class RemoteClusterAdd extends PureComponent {
defaultMessage="Add remote cluster"
/>
}
description={
<FormattedMessage
id="xpack.remoteClusters.remoteClustersDescription"
defaultMessage="Add a remote cluster that is connected to either seed nodes or a single proxy address."
/>
}
/>

<RemoteClusterForm
Expand Down
2 changes: 1 addition & 1 deletion ...ns/remote_clusters/public/application/sections/remote_cluster_edit/remote_cluster_edit.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ export class RemoteClusterEdit extends Component {
>
<FormattedMessage
id="xpack.remoteClusters.edit.deprecatedSettingsMessage"
defaultMessage="This remote cluster contains deprecated settings. We have attempted to resolve them. Please verify all changes before saving."
defaultMessage="This remote cluster has deprecated settings that we tried to resolve. Verify all changes before saving."
/>
</EuiCallOut>
<EuiSpacer />
Expand Down
60 changes: 35 additions & 25 deletions ...ote_clusters/public/application/sections/remote_cluster_list/detail_panel/detail_panel.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ export class DetailPanel extends Component {
title={
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsTitle"
defaultMessage='"{remoteCluster}" contains deprecated settings'
defaultMessage="'{remoteCluster}' has deprecated settings"
values={{
remoteCluster: clusterName,
}}
Expand All @@ -134,28 +134,38 @@ export class DetailPanel extends Component {
color="warning"
iconType="help"
>
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsMessage"
defaultMessage="We recommend updating this remote cluster to use the correct settings. {helpLink}"
values={{
helpLink: isConfiguredByNode ? (
// A remote cluster is not editable if configured in elasticsearch.yml, so we direct the user to documentation instead
<EuiLink href={proxyModeUrl} target="_blank">
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsLearnMoreLinkLabel"
defaultMessage="Learn more."
/>
</EuiLink>
) : (
<EuiLink {...getRouterLinkProps(`${CRUD_APP_BASE_PATH}/edit/${clusterName}`)}>
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsEditLinkLabel"
defaultMessage="Edit remote cluster."
/>
</EuiLink>
),
}}
/>
{/* A remote cluster is not editable if configured in elasticsearch.yml, so we direct the user to documentation instead */}
{isConfiguredByNode ? (
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsConfiguredByNodeMessage"
defaultMessage="Edit the cluster to update the settings. {helpLink}"
values={{
helpLink: (
<EuiLink href={proxyModeUrl} target="_blank">
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsLearnMoreLinkLabel"
defaultMessage="Learn more."
/>
</EuiLink>
),
}}
/>
) : (
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsMessage"
defaultMessage="{editLink} to update the settings."
values={{
editLink: (
<EuiLink {...getRouterLinkProps(`${CRUD_APP_BASE_PATH}/edit/${clusterName}`)}>
<FormattedMessage
id="xpack.remoteClusters.detailPanel.deprecatedSettingsEditLinkLabel"
defaultMessage="Edit the cluster"
/>
</EuiLink>
),
}}
/>
)}
</EuiCallOut>
<EuiSpacer size="l" />
</>
Expand Down Expand Up @@ -249,7 +259,7 @@ export class DetailPanel extends Component {
<EuiTitle size="xs">
<FormattedMessage
id="xpack.remoteClusters.detailPanel.maxConnectionsPerClusterLabel"
defaultMessage="Maximum number of connections"
defaultMessage="Maximum connections"
/>
</EuiTitle>
</EuiDescriptionListTitle>
Expand Down Expand Up @@ -363,7 +373,7 @@ export class DetailPanel extends Component {
<EuiTitle size="xs">
<FormattedMessage
id="xpack.remoteClusters.detailPanel.maxSocketConnectionsLabel"
defaultMessage="Maximum number of socket connections"
defaultMessage="Maximum socket connections"
/>
</EuiTitle>
</EuiDescriptionListTitle>
Expand Down
2 changes: 1 addition & 1 deletion ...lic/application/sections/remote_cluster_list/remote_cluster_table/remote_cluster_table.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ export class RemoteClusterTable extends Component {
content={
<FormattedMessage
id="xpack.remoteClusters.remoteClusterList.table.hasDeprecatedSettingMessage"
defaultMessage="This remote cluster contains deprecated settings. Edit the remote cluster to resolve."
defaultMessage="Edit this cluster to update deprecated settings."
/>
}
/>
Expand Down